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

[Fix] #409 - 스프린트0 QA #411

Merged
merged 3 commits into from
Oct 17, 2024
Merged

[Fix] #409 - 스프린트0 QA #411

merged 3 commits into from
Oct 17, 2024

Conversation

dlwogus0128
Copy link
Contributor

@dlwogus0128 dlwogus0128 commented Oct 17, 2024

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

스프린트0 QA 내용들을 반영했어요 후후

  • 오늘의 솝마디 결과뷰 -> 콕찌르기 시 토스트바 노출
  • 오늘의 솝마디 결과뷰 -> 콕찌르기 추천인이 없을 시의 예외 처리

📌 참고 사항

오늘의 솝마디 결과뷰 -> 콕찌르기 추천인이 없을 시의 예외 처리

위의 상황은 이번에 QA하면서, 모든 멤버에게 콕찌르기를 진행할 경우 추천인이 오지 않아 작동이 멈추는 버그였는데요.
이게 dev라 멤버 인원이 적어서 가능한 것이지, prod에서는 모든 멤버에게 콕찌르기 하는 것이 사실상 불가능한 완전완전 엣지 케이스이기 때문에 ..
배포해도 큰 문제는 없을 것 같기도 하고, 추천인이 없을 경우 '모든 솝트인에게 찔렀어요' 엠티뷰 혹은 특별한 이스터에그를 기획적으로 진행하고 싶어 하셔서, 해당 내용은 다음 스프린트로 미뤘습니다. (그래도 예외처리는 해뒀습니다..)

📸 스크린샷

기능 스크린샷
(엣지 케이스) 추천인 없을 경우
콕찌르기 시 토스트바
Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-10-17.at.19.52.33.mp4

📮 관련 이슈

@dlwogus0128 dlwogus0128 added Fix 문제 해결, 코드 수정 재현✦ labels Oct 17, 2024
@dlwogus0128 dlwogus0128 requested a review from yungu0010 October 17, 2024 10:42
@dlwogus0128 dlwogus0128 self-assigned this Oct 17, 2024
Copy link

height bot commented Oct 17, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Contributor

@yungu0010 yungu0010 left a comment

Choose a reason for hiding this comment

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

깔끔하다. . 🍀 짱 !

@dlwogus0128 dlwogus0128 merged commit 503c561 into develop Oct 17, 2024
@dlwogus0128 dlwogus0128 deleted the fix#409-sprint0-qa branch October 17, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 문제 해결, 코드 수정 size/S 재현✦
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 스프린트0 QA 반영
2 participants