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

골 수락 기능 구현 #85

Closed
wants to merge 10 commits into from
Closed

골 수락 기능 구현 #85

wants to merge 10 commits into from

Conversation

jhsseonn
Copy link
Member

@jhsseonn jhsseonn commented Mar 9, 2024

3/10
골 수락 기능 구현 완료했습니다. 이번주까지 시험이 있어서 진도를 많이 못나갔습니다.. 다음주부터 속도 내보겠습니다!
[주요 작업 내용]

  • 골 초대 수락
    • 골 관리자의 경우 골 생성시에 자동으로 수락됨으로 처리
    • 골 참여자가 아니거나 골 참여자이지만 수락을 하지 않은 사용자는 단일 골 조회를 할 수 없도록 제한
  • closed [Feat] 골 수락 기능 구현 #58

[질문]

  1. 골 수락 기능 서비스 테스트를 하는데 로직에는 문제가 없는데 테스트에는 반영이 제대로 되지 않아서 혹시 트랜잭션 문제인가 싶어 @Transactional 을 테스트 메서드에 부여해주니 문제없이 잘 돌아가더라구요.. 이전까지 수정 기능 테스트하면서 트랜잭션 문제가 있었던 적이 없었는데 이번에 처음 발생한 문제라 정확한 원인을 좀 더 찾아보도록 하겠습니다. 우선 테스트 상으로는 문제가 없어 그대로 둘지 아니면 수정하는 것이 좋을지 질문드립니다!

@jhsseonn jhsseonn added the feature 기능 추가 label Mar 9, 2024
@jhsseonn jhsseonn self-assigned this Mar 9, 2024
@jhsseonn jhsseonn closed this Mar 9, 2024
@jhsseonn jhsseonn deleted the feature/58 branch March 9, 2024 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 추가
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Feat] 골 수락 기능 구현
1 participant