-
Notifications
You must be signed in to change notification settings - Fork 7
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
Feat: 모달창 추가, 예약상태 추가 #116
base: dev
Are you sure you want to change the base?
Conversation
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.
넘넘 수고해써 ⭐️⭐️⭐️⭐️⭐️
예약할 때 어떤 시간을 클릭하고 그보다 더 이전 시간 클릭하면 (예를 들어 16:00-16:30 을 클릭하고 12:00-12:30 을 클릭) 완전히 화면이 멈추면서 오류가 나는데 이것만 해결하면 될듯!!
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.
상태관리 괜찮은 것 같은데 !!! 컴포넌트 분리도 잘 해두고 네이밍도 직관적이구 해서 좋은 것 같아용
components/reservation/TimeTable.tsx
Outdated
// 예약 상태 타입 | ||
export interface Reservation { | ||
timeRange: string; | ||
status: 'unavailable' | 'booked' | 'myReservation' | 'available'; | ||
name?: string; // 예약자의 이름 (예약 마감 상태일 때만 표시) | ||
} |
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.
reservation interface 만들어두는거 완전 굳인데 앞에서 한 리뷰처럼 dto에 맞게 수정해두면 좋을듯 !!
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.
수고하셨습니다!
한 가지 이슈 발견하여 영상과 함께 첨부합니다.
1시간 이상 선택하는 경우에
영상 1처럼 첫시간 ~ 마지막 시간을 선택하면 중간이 채워지지만, 영상 2처럼 한시간을 이미 선택한 후 추가적으로 시간을 선택하는 경우에 기존에 선택한 내용이 삭제됩니다. 이 부분 확인해 주세요!
#️⃣연관된 이슈
📝작업 내용
( 4. 예약 정보를 담는 Reservation 인터페이스 생성해두었는데, 이 부분은 API 연결하면서 수정될 것 같습니다.)
🔎코드 설명 및 참고 사항
💬리뷰 요구사항