Skip to content
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

Merged
merged 9 commits into from
Oct 4, 2023

Conversation

falconlee236
Copy link
Member

Summary

  • 멘토링 취소 모달인 CancelModal을 추가했습니다.
  • 홈 화면에 Feedback 모달을 추가했습니다.

Description

  1. 멘토링 취소 모달인 CancelModal을 추가했습니다.
    해당 모달은 components/Cancel/CancelModal.tsx에 위치해 있습니다.
    일단 취소 모달의 Select 항목의 내용은 기존 [todo] 멘토링 취소 모달 개발 #190 에서 말해주셨던 항목을 참고했습니다.
    해당 모달은 page/index.tsx, page/Mentorings/index.tsx, page/reservation/[reservationId].tsx에 추가했습니다.
  2. 홈 화면에 Feedback 모달을 추가했습니다.
    @jinaji 님이 [feat]: reservation page #223 #230 에서 말씀하셨던 [todo] home 에서 reservation 의 하위 모달도 뜰 수 있도록 변경 #207 이슈를 처리했습니다.

참고사항

취소 모달의 Select항목의 추가 아이디어가 안떠올라서 일단 3개만 했는데, 추가적으로 넣으면 좋을 것같다는 의견 있으면 남겨주세요

ScreenShot

image image

Copy link
Contributor

@jinaji jinaji left a 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); 부분이 신기해서 한참 봤네여 ...
오탈자 관련 코멘트 달아둔 부분 혹시 한 번 확인 가능하시면 수정 부탁드려도 될까여!!
저에게 어사인 걸렸던 이슈 ... 해결해주셔서 감사합니다 고생하셨어여~!

42manito/src/components/Cancel/CancelModal.tsx Outdated Show resolved Hide resolved
Copy link
Member

@JuneParkCode JuneParkCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

몇가지 수정사항 있습니다.

42manito/src/components/Cancel/CancelModal.tsx Outdated Show resolved Hide resolved
42manito/src/components/Cancel/CancelModal.tsx Outdated Show resolved Hide resolved
@falconlee236 falconlee236 merged commit 93ea4d8 into manito42:dev Oct 4, 2023
1 check passed
Copy link
Contributor

@obvoso obvoso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰 달았던거 pending상태로 submit이 안되었었네여... ㅋ.ㅋ
확인 했습니다!

42manito/src/components/Cancel/CancelModal.tsx Outdated Show resolved Hide resolved
obvoso pushed a commit to obvoso/42manito-front that referenced this pull request Oct 27, 2023
* [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 초기값 수정
@falconlee236 falconlee236 linked an issue Mar 6, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request fix 수정 작업
Projects
None yet
4 participants