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

[HOTFIX] API가 중복 호출되어 QNA 리스트가 7개를 넘어가는 오류 해결 #99

Merged
merged 1 commit into from
Aug 23, 2023
Merged

[HOTFIX] API가 중복 호출되어 QNA 리스트가 7개를 넘어가는 오류 해결 #99

merged 1 commit into from
Aug 23, 2023

Conversation

ddongseop
Copy link
Member

📌 관련 이슈

✨ 어떤 이유로 변경된 내용인지

  • 간혹 온보딩 끝나는 측에서 버튼을 2번 누르는 바람에 API가 2번 호출되는 경우 오류 발생
  • QNA 리스트의 크기가 1+6개가 아닌, 1+6+6개가 되기 때문에 이 부분을 에러 핸들링 하는 것이 필요하다고 판단

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

  • 아라가 매칭 해달라고 해서, 테스트 해보다가 찾은 오류여서 바로 해결합니닷!

@ddongseop ddongseop added 🔥 Pull Request PR 날림 🚨Hotfix 급한 수정사항 발생 ddongseop🍑 labels Aug 23, 2023
@ddongseop ddongseop self-assigned this Aug 23, 2023
jun02160
jun02160 approved these changes Aug 23, 2023
View reviewed changes
Copy link
Member

@jun02160 jun02160 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

@ddongseop ddongseop merged commit ee39c8c into develop Aug 23, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jun02160 jun02160 jun02160 approved these changes

Assignees

@ddongseop ddongseop

Labels
ddongseop🍑 🚨Hotfix 급한 수정사항 발생 🔥 Pull Request PR 날림
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ddongseop @jun02160