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.

Sign up for GitHub

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

Jump to bottom

[FE] feat: 리뷰 작성 페이지에 변경된 리뷰 작성 페이지의 API 반영 #317

Merged
merged 7 commits into from
Aug 12, 2024
Merged

[FE] feat: 리뷰 작성 페이지에 변경된 리뷰 작성 페이지의 API 반영 #317

merged 7 commits into from
Aug 12, 2024

Conversation

BadaHertz52
Copy link
Contributor

🚀 어떤 기능을 구현했나요 ?

  • 리뷰 작성 페이지에 변경된 리뷰 작성 페이지의 API를 반영했어요.

🔥 어떻게 해결했나요 ?

  • 관련된 훅을 생성했어요.
  • AnswerListPreviewModal은 AnswerListRecheckModal로 네이밍을 변경했어요.
  • AnswerListRecheckModal이 답변 내용을 가지도록 수정했어요. 답변 내용을 통해 선택된 객관식 문항, 작성된 서술형 문항을 보여주도록 하고 있어요.
  • 서버의 baseurl 변경에 따른 endpoint를 변경했어요.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 빠른 시간에 완성을 해서 빠진 부분이 없는지 확인해주세요.

📚 참고 자료, 할 말

  • 어휴...

@BadaHertz52
feat: 리뷰 작성 카드 폼에서 리뷰 작성을 위한 데이터 받아오는 api 훅, 목 핸들러 생성
3ddb79c 
- ReviewWritingCardFormPage 의 path 경로 변경
- 리뷰 작성 카드 폼에서 리뷰 작성을 위한 데이터 받아오는 api 훅, 목 핸들러 생성
- gettingDataToWritingReview 의 endpoint 변경
@BadaHertz52
feat: 리뷰 작성 카드 폼에 제출 모달, 오류 모달, 미리보기 모달 적용 및 AnswerListPrevieModal 수정
967da13
@BadaHertz52
refactor: 불필요한 파일 삭제 및 mockData 관련 경로 정리
e8d9c5c
@BadaHertz52
feat: 첫 번째 카드에서 이전 버튼 감추기
fb42738
@BadaHertz52
refactor: 미리보기를 '제출 전 확인'으로 수정
1f483c8 
- AnswerListPreviewModal -> AnswerListRecheckModal
- PreviewButton -> RecheckButton
@BadaHertz52
chore: api base url 변경에 따른 endpoint, handler 수정
a5f6ea7
@BadaHertz52
chore: 오타 수정
3e7c90d
@BadaHertz52 BadaHertz52 added this to the 4차 스프린트: 최종 milestone Aug 12, 2024
@BadaHertz52 BadaHertz52 self-assigned this Aug 12, 2024
@BadaHertz52 BadaHertz52 linked an issue Aug 12, 2024 that may be closed by this pull request
[FE] 리뷰 작성 페이지에 변경된 리뷰 작성 페이지의 API를 반영한다. #303
Closed
ImxYJL
ImxYJL approved these changes Aug 12, 2024
View reviewed changes
Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

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

빠진 건...빠졌을 때 다시 살펴봅시다 ㅋㅋㅋㅋㅋ

soosoo22
soosoo22 approved these changes Aug 12, 2024
View reviewed changes
Copy link
Contributor

@soosoo22 soosoo22 left a comment

Choose a reason for hiding this comment

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

리뷰 작성 페이지 api 반영하느라 수고했어요!!

chysis
chysis approved these changes Aug 12, 2024
View reviewed changes
Copy link
Contributor

@chysis chysis left a comment

Choose a reason for hiding this comment

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

너무 고생이 많아요🔥

@BadaHertz52 BadaHertz52 merged commit ec840ab into develop Aug 12, 2024
5 checks passed
@BadaHertz52 BadaHertz52 changed the title [FE] 리뷰 작성 페이지에 변경된 리뷰 작성 페이지의 API 반영 [FE] feat: 리뷰 작성 페이지에 변경된 리뷰 작성 페이지의 API 반영 Aug 12, 2024
@donghoony donghoony deleted the fe/feat/303-review_writing_form_api branch August 20, 2024 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chysis chysis chysis approved these changes

@soosoo22 soosoo22 soosoo22 approved these changes

@ImxYJL ImxYJL ImxYJL approved these changes

Assignees

@BadaHertz52 BadaHertz52

Labels
🍀 FE 🚀 기능 🫠 힘들다...
Projects
2024-review-me
Status: Done
Milestone
4차 스프린트: 최종
Development

Successfully merging this pull request may close these issues.

[FE] 리뷰 작성 페이지에 변경된 리뷰 작성 페이지의 API를 반영한다.
4 participants
@BadaHertz52 @chysis @soosoo22 @ImxYJL