feat: eventName 필드 추가 - #199

[sjk4618]

🔥Pull requests

⛳️ 작업한 브랜치

• Resolved: feat: 행사 siteMap 조회 api 구현 및 siteMap 등록 추가 #199

👷 작업한 내용

🚨 참고 사항

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

릴리스 노트

• 개선 사항
  • 이벤트 사이트맵 이미지 조회 응답에 이벤트 이름 정보가 추가되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

개요

EventSiteMapGetResponse 레코드에 eventName 필드를 추가하고, 이에 따라 팩토리 메서드 및 서비스 구현을 업데이트하여 응답에 이벤트 이름을 포함하도록 변경했습니다.

변경사항

응집 / 파일	변경 요약
응답 DTO 업데이트 src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/dto/res/EventSiteMapGetResponse.java	공개 레코드에 String eventName 필드 추가 및 팩토리 메서드 of() 시그니처를 of(List<EventSiteMapImageInfo>) → of(final String eventName, final List<EventSiteMapImageInfo> images)로 업데이트
서비스 구현 수정 src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java	getEventSiteMapImages 메서드에서 EventSiteMapGetResponse.of() 호출 시 eventName 파라미터 추가

예상 코드 리뷰 소요 시간

🎯 2 (단순) | ⏱️ ~10분

• 두 파일 간 일관된 변경 패턴 확인
• eventName 파라미터가 서비스에서 올바르게 전달되는지 검증
• 기존 호출 지점에서 새로운 메서드 시그니처가 모두 업데이트되었는지 확인

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Linked Issues check	❓ Inconclusive	PR의 변경 사항(eventName 필드 추가)은 이슈 #199의 행사 siteMap 조회 API 구현의 일부로 보이나, 전체 요구 사항 충족 여부가 명확하지 않습니다.	이슈 #199의 전체 요구 사항(siteMap 조회 API 구현 및 등록 기능)에 대한 완전한 구현 여부를 확인하세요.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목은 PR의 주요 변경 사항인 eventName 필드 추가를 명확하게 설명하고 있으며, 관련 이슈 번호도 포함되어 있습니다.
Description check	✅ Passed	설명은 작업한 브랜치 정보와 해당 이슈 번호를 언급하고 있어 변경 사항과 관련이 있습니다.
Out of Scope Changes check	✅ Passed	모든 변경 사항은 eventName 필드 추가 및 해당 메서드 시그니처 업데이트로 이슈 #199와 관련된 범위 내에 있습니다.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#199

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 20e6423 and 57859c1.

📒 Files selected for processing (2)

• src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/dto/res/EventSiteMapGetResponse.java (1 hunks)
• src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java (1 hunks)

🧰 Additional context used

🧠 Learnings (1)

📓 Common learnings

Learnt from: sjk4618
Repo: PERMIT-SEOUL/permit-server PR: 48
File: src/main/java/com/permitseoul/permitserver/domain/guest/core/domain/entity/GuestEntity.java:31-37
Timestamp: 2025-07-15T09:37:32.765Z
Learning: sjk4618 prefers to implement factory methods or public constructors for entities when they are actually needed, rather than creating them proactively.

🔇 Additional comments (2)

src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java (1)

39-39: LGTM!

이미 조회된 event 객체에서 getName()을 호출하여 응답에 포함시키는 것은 적절한 구현입니다. 추가적인 쿼리 없이 필요한 데이터를 활용하고 있습니다.

src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/dto/res/EventSiteMapGetResponse.java (1)

6-12: LGTM! 단, API 호환성을 확인해 주세요.

eventName 필드 추가와 factory method 시그니처 변경은 깔끔하게 구현되었습니다. 기존 API를 사용하는 클라이언트가 있다면 응답 구조 변경에 따른 호환성 영향을 확인해 주세요.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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