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

[FE] fix : 필수가 아닌 답변을 작성하려 하지 않았을 경우에 대한 오류 수정 #342

Merged
merged 2 commits into from
Aug 13, 2024

Conversation

BadaHertz52
Copy link
Contributor


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

🔥 어떻게 해결했나요 ?

  • 답변을 작성할때, answerMap, answerValidationMap 의 상태를 변경하기 때문에 작성하지 않은 필수가 아닌 답변의 값이 안들어가는 오류가 있었어요.
  • questionList 가 변경될때, 답변의 기본값, 답변의 유효성 검사 기본값을 설정하게 했어요.
  • 답변의 기본값에 기존에 작성한 답이 있을 경우 이를 반영하도록 했어요.

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

  • 코드 리뷰 시 프론트엔드가 여러 가지 경우를 같이 확인해봤으면 좋겠어요.

📚 참고 자료, 할 말

  • 오류 수정,....오류...오류 수정... 엣지 케이스 파악 잘 하고 싶다.

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.

엣지 케이스 멈춰!!!

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.

변경된 점 확인했어요 :) 고생했어요!👍

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.

오류 수정하느라 고생했어요!!

@BadaHertz52 BadaHertz52 merged commit b0ab7d5 into develop Aug 13, 2024
5 checks passed
@donghoony donghoony deleted the fe/fix/341_review_writing_not_required_error branch August 20, 2024 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] 필수가 아닌 답변을 작성하려 하지 않았을 경우에 대한 오류를 수정한다.
4 participants