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: 나갔던 약속에 재참여가 되지 않는 문제 개선 #880

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

coli-geonwoo
Copy link
Contributor

@coli-geonwoo coli-geonwoo commented Nov 4, 2024

🚩 연관 이슈

close #879


📝 작업 내용

  • meeting_member unique 조건 > meeting_member_deleteAt unique 조건으로 변경

기존에 약속-회원의 유니크 조건으로인해 퇴장했던 약속에 재참여가 불가하다는 이슈가 발생했습니다.
그렇다고 이 조건을 해제해버리면 동일 약속에 회원이 무한대로 재참여할 수 있게 되는 문제가 발생합니다.

따라서 유니크 조건에 삭제 여부를 같이 걸어서 퇴장했던 약속의 재참여를 가능하게 하되 퇴장하지 않은 참여자의 경우 재참여를 불가하도록 설정하였습니다.


🏞️ 스크린샷 (선택)


🗣️ 리뷰 요구사항 (선택)

테스트는 유저 행동 시나리오라고 생각하여 dynamicTest로 인수 테스트를 작성하였습니다.
불필요한 작업이라 생각되면 단위 테스트로 바꿀 예정이니 의견 남겨주세요.

Copy link

github-actions bot commented Nov 4, 2024

Test Results

 48 files   48 suites   6s ⏱️
178 tests 178 ✅ 0 💤 0 ❌
179 runs  179 ✅ 0 💤 0 ❌

Results for commit 30c968c.

Copy link

github-actions bot commented Nov 4, 2024

📝 Test Coverage Report

Overall Project 82.03% 🍏
File Coverage
Mate.java 100% 🍏

Copy link
Contributor

@hyeon0208 hyeon0208 left a comment

Choose a reason for hiding this comment

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

[제안]
우리 서비스 최고 VIP 콜리 ~ 빠른 반영 감사합니다

해음이 약속 방 나간 후 재참여 여부에 대해 이야기 해보자고 했었는데
추후에 한 번더 이야기 해보면 좋을 것 같아요 👍

해당 플로우에 대해 BDD로 테스트를 작성하니 오히려 가독성이 좋은 것 같아요 👍

Copy link
Contributor

@eun-byeol eun-byeol left a comment

Choose a reason for hiding this comment

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

슬랙 보고 유니크키 조건 빼버린줄 알고 말벌아저씨마냥 달려왔는데
아니었네요ㅎㅎ 구웃~

동적 테스트도 유저 시나리오 관점에서 합리적이라고 생각합니다~!
빠르게 버그픽스해주어 감사합니다🙇‍♀️

@eun-byeol
Copy link
Contributor

[제안]
PR 메시지에 들어가는 이미지가 너무 커서 올릴때 html로 비율 줄여주면 더 좋을 것 같아요! (제가 변경해두었습니다~)
<img src="https://github.com/user-attachments/assets/28d91c7c-9b45-4ed5-be4b-c667adc028c6" width="50%" height="auto">

Copy link
Contributor

@mzeong mzeong left a comment

Choose a reason for hiding this comment

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

이렇게 하면 모임방을 나갔다가 다시 들어온 후 다시 나가려고 하면 나갈 수가 없지 않나요? deletedAt이 boolean 값이 아니라 타임이라 상관 없겠네요

Copy link
Contributor

@mzeong mzeong left a comment

Choose a reason for hiding this comment

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

유니크 키 삭제하는 방향보다 좋아요 👍👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 나간 약속에 재참여가 되지 않는 문제 해결
4 participants