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] : 스타디움 구역 정보 중복 & 구역 추천(KT) 문제를 해결한다 #89

Merged
merged 3 commits into from
Nov 23, 2024

Conversation

bbbang105
Copy link
Member

@bbbang105 bbbang105 commented Nov 23, 2024

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

스타디움 구역 정보 중복 문제 해결

    private List<GetStadiumInfosResponseDto.ZoneInfo> getZonesNameAndColorFromStadium(StadiumStatusType[] statusTypes) {
        return Arrays.stream(statusTypes)
                .filter(status -> status != CHEERING_DUMMY)
                .map(status -> GetStadiumInfosResponseDto.ZoneInfo.of(status.getZoneName(), status.getZoneColor()))
                .toList();
    }
  • CHEERING_DUMMY는 filter로 제외

구역 추천(KT) 문제 해결

  • 배포 DB에 중복 데이터 존재 -> 제거함으로써 해결

✏️ 관련 이슈

본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요


🎸 기타 사항 or 추가 코멘트

@bbbang105 bbbang105 added 🚨 fix 버그 수정 / 에러 해결 🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 🤯 SANGHO 상호 Issue or PR labels Nov 23, 2024
@bbbang105 bbbang105 requested a review from juuuunny November 23, 2024 19:34
@bbbang105 bbbang105 self-assigned this Nov 23, 2024
Copy link
Contributor

@juuuunny juuuunny left a comment

Choose a reason for hiding this comment

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

버그 수정 좋습니다!!

@bbbang105 bbbang105 merged commit c82c786 into develop Nov 23, 2024
1 check passed
@bbbang105 bbbang105 deleted the fix/#88/duplicate-zone branch November 23, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 🚨 fix 버그 수정 / 에러 해결 🤯 SANGHO 상호 Issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] : 스타디움 구역 정보가 중복되어 나오는 문제를 해결한다
2 participants