-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FE] 일정 등록 모달 기본 날짜 변경 #245
The head ref may contain hidden characters: "refactor/fe/\uC77C\uC815-\uB4F1\uB85D-\uBAA8\uB2EC-\uAE30\uBCF8-\uB0A0\uC9DC-\uBCC0\uACBD"
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,15 +9,16 @@ import useScheduleAddModal from '~/hooks/schedule/useScheduleAddModal'; | |
|
||
interface ScheduleAddModalProps { | ||
teamPlaceName: string; | ||
clickedDate: Date; | ||
} | ||
|
||
const ScheduleAddModal = (props: ScheduleAddModalProps) => { | ||
const { teamPlaceName } = props; | ||
const { teamPlaceName, clickedDate } = props; | ||
const { closeModal } = useModal(); | ||
const { | ||
schedule, | ||
handlers: { handleScheduleChange, handleScheduleSubmit }, | ||
} = useScheduleAddModal(); | ||
} = useScheduleAddModal(clickedDate); | ||
|
||
return ( | ||
<Modal> | ||
|
@@ -55,7 +56,7 @@ const ScheduleAddModal = (props: ScheduleAddModalProps) => { | |
<Input | ||
width="220px" | ||
height="40px" | ||
type="datetime-local" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 시간 선택이 없어졌는데,, 시간 선택은 있어야 할 것 같아 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 응 일단 날짜만 수정했고, Menu 컴포넌트 이용해서 이어붙이는 식으로 구현할거 같아 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 나중에라도 꼭 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 음 아마 date로 날짜 선택 > 타임 테이블 메뉴에서 시간 선택 > 데이터 취합 및 가공해서 api 요청 이런 식으로 수정할 예정이라 date로 수정했어! 부연 설명이 부족했네 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 좋아요 그러면 지금 할거아니면 이슈파서 유스가 그대로 이어가는게 좋아보여! 이슈 작성하고 유스가 머지해줘 어프루브는 해둘게! |
||
type="date" | ||
css={S.dateTimeLocalInput} | ||
name="startDateTime" | ||
value={schedule['startDateTime']} | ||
|
@@ -74,10 +75,11 @@ const ScheduleAddModal = (props: ScheduleAddModalProps) => { | |
<Input | ||
width="220px" | ||
height="40px" | ||
type="datetime-local" | ||
type="date" | ||
css={S.dateTimeLocalInput} | ||
name="endDateTime" | ||
value={schedule['endDateTime']} | ||
min={schedule['startDateTime']} | ||
onChange={handleScheduleChange} | ||
required | ||
/> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이게 꼭 필요한가? DateCell은 Date에 대한 정보를 가지고 있잖아!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 그러네? 반영해씀다