Skip to content

Conversation

@7ijin01
Copy link
Member

@7ijin01 7ijin01 commented Sep 16, 2025

📌 PR 개요


✅ 변경사항


🔍 체크리스트

  • PR 제목은 명확한가요?
  • 관련 이슈가 있다면 연결했나요?
  • 로컬 테스트는 통과했나요?
  • 코드에 불필요한 부분은 없나요?

📎 관련 이슈

Closes #89


💬 기타 참고사항

Summary by CodeRabbit

  • 신기능
    • 마이페이지에서 사용자 관광지 저장 시 기존 개수 제한이 해제되었습니다. 이제 중복 등록만 방지되며 원하는 만큼 관광지를 추가할 수 있습니다. 화면과 사용 흐름은 동일하고, 기존에 저장된 항목에도 영향이 없습니다. 추가·삭제·보기 기능은 그대로 동작하며, 저장 시 오류 알림은 중복 등록인 경우에만 표시됩니다.

@7ijin01 7ijin01 linked an issue Sep 16, 2025 that may be closed by this pull request
@coderabbitai
Copy link

coderabbitai bot commented Sep 16, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

사용자 선호 관광지 저장 로직에서 5개 제한 검사와 예외(TOURSPOT_EXCEEDS)를 제거했다. 중복 검사(TOURSPOT_ALREADY_EXISTS)는 유지되며, 존재 확인 후 저장하는 흐름은 동일하다. 공개 메서드 시그니처 변화는 없다.

Changes

Cohort / File(s) Summary
MypageService 변경
src/main/java/com/opendata/domain/mypage/service/MypageService.java
saveUserTourSpot에서 사용자 보유 수 5개 초과 검사 및 관련 예외 제거. 중복 체크 유지, 나머지 저장 절차는 변경 없음. 공개 API 시그니처 불변.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant C as Client
  participant S as MypageService
  participant R as Repository

  C->>S: saveUserTourSpot(userId, tourSpotId)
  S->>R: existsByUserAndTourSpot(userId, tourSpotId)
  R-->>S: exists? (true/false)
  alt 중복일 때
    S-->>C: throw TOURSPOT_ALREADY_EXISTS
  else 신규일 때
    S->>R: save(TourSpotComponent)
    R-->>S: saved entity
    S-->>C: return saved
  end
  note over S: 이전 흐름의 "5개 제한 검사" 단계가 제거됨
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

토끼 발로 톡톡, 다섯 셈은 이제 놔요
마음 가는 관광지, 주머니에 다 넣어요
겹치지만 않으면, 펄짝 저장 완료!
달빛 지도 펼치고, 더 멀리 떠나볼까요
힙합 귀 흔들며—좋아요가 늘어나요 🐇✨

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/#89-preference-exceed

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b75df5e and 987aa41.

📒 Files selected for processing (1)
  • src/main/java/com/opendata/domain/mypage/service/MypageService.java (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@7ijin01 7ijin01 merged commit 42a3681 into main Sep 16, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] 선호 관광지 조회 5개 제한 해제

2 participants