-
Notifications
You must be signed in to change notification settings - Fork 6
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
[todo] 멘토링 취소 모달 개발 #235
[todo] 멘토링 취소 모달 개발 #235
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.
홈, 멘토링 페이지, 예약 페이지에서 작동 잘 되고 메일로 사유 날아오는 것 확인했습니다!
디폴트 메세지 요정도면 된 것 같기도 하고 ... 저는 괜찮아보여용
defaultMsg.find(({value: id}) => id === value); 부분이 신기해서 한참 봤네여 ...
오탈자 관련 코멘트 달아둔 부분 혹시 한 번 확인 가능하시면 수정 부탁드려도 될까여!!
저에게 어사인 걸렸던 이슈 ... 해결해주셔서 감사합니다 고생하셨어여~!
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.
몇가지 수정사항 있습니다.
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.
리뷰 달았던거 pending상태로 submit이 안되었었네여... ㅋ.ㅋ
확인 했습니다!
* [fix]: index.tsx 메인 페이지에 Cancel Modal 적용 * [fix]: Mentoring index.tsx에 Cancel Modal 적용 * [fix]: Reservation page에 Cancel Modal 적용 * [fix]: Home page에 Feedback Modal 적용 manito42#157 * [fix]: CANCLE읋 CANCEL로 변환 * [fix]: Success, Fail 문자열 변경 * [fix]: TextArea 에서 최대 문자 길이 제한 변경 * [fix]: useState 초기값 수정
Summary
Description
해당 모달은 components/Cancel/CancelModal.tsx에 위치해 있습니다.
일단 취소 모달의 Select 항목의 내용은 기존 [todo] 멘토링 취소 모달 개발 #190 에서 말해주셨던 항목을 참고했습니다.
해당 모달은 page/index.tsx, page/Mentorings/index.tsx, page/reservation/[reservationId].tsx에 추가했습니다.
@jinaji 님이 [feat]: reservation page #223 #230 에서 말씀하셨던 [todo] home 에서 reservation 의 하위 모달도 뜰 수 있도록 변경 #207 이슈를 처리했습니다.
참고사항
취소 모달의 Select항목의 추가 아이디어가 안떠올라서 일단 3개만 했는데, 추가적으로 넣으면 좋을 것같다는 의견 있으면 남겨주세요
ScreenShot