search

woowacourse-teams / 2024-mouda

17 stars 6 forks source link

안내면진다 조회시 당첨자 가져오도록 수정 #602

Closed ay-eonii closed 2 weeks ago

ay-eonii commented 2 weeks ago

PR의 목적이 무엇인가요?

안내면진다 상세조회 API에서 isAnnounced 필드가 항상 false 인 문제를 해결했습니다.

이슈 ID는 무엇인가요?

설명

Builder를 사용하며 필드를 놓친 게 원인이었습니다. 기존 인덴트가 달라 변경사항이 큰데 수정 지점은 아래 하나입니다

변경 후

return Bet.builder()
    .betDetails(betEntity.toBetDetails())
    .moimerId(betEntity.getMoimerId())
    .participants(participants)
    .loserId(betEntity.getLoserDarakbangMemberId())
    .build();

기존에 빠진 loseId 를 추가했습니다.

질문 혹은 공유 사항 (Optional)