[LNK-32] 생일 알림 설정 조회 추가

[jj0526]

Related issue 🛠

• x

작업 내용 💻

• 사용자 알림 설정 조회에 생일 알림 허용 여부 추가하였습니다

스크린샷 📷

• 테스트

    "code": 1110,
    "message": "알림 설정 조회에 성공했습니다.",
    "data": {
        "isNewLeenkNotify": false,
        "isLeenkStatusNotify": true,
        "isNewFeedNotify": true,
        "isNewReactionNotify": true,
        "isBirthdayNotify": true
    }
}

같이 얘기해보고 싶은 내용이 있다면 작성 📢

• x

Summary by CodeRabbit

릴리스 노트

• 새로운 기능
  • 알림 설정에서 생일 알림을 개별적으로 관리할 수 있는 기능이 추가되었습니다.

[coderabbitai]

개요

생일 알림 기능을 지원하기 위해 NotificationSettingResponse 응답 객체에 isBirthdayNotify 필드를 추가하고, 이를 매핑하는 로직을 UserSettingMapper에서 업데이트했습니다.

변경 사항

코호트 / 파일	변경 요약
알림 설정 응답 객체 src/main/java/leets/leenk/domain/user/application/dto/response/NotificationSettingResponse.java	isBirthdayNotify 필드 추가 (스키마 주석 포함). isNewReactionNotify 필드에 후행 쉼표 추가
사용자 설정 매퍼 src/main/java/leets/leenk/domain/user/application/mapper/UserSettingMapper.java	toNotificationSettingResponse 메서드에서 isBirthdayNotify 매핑 로직 추가

예상 코드 리뷰 난이도

🎯 2 (Simple) | ⏱️ ~8 분

• 필드 추가 및 매핑 로직이 기존 패턴을 따르고 있는지 확인
• 스키마 주석의 정확성 검토 (설명: "생일 알림", 예제: "true")

추천 리뷰어

• hyxklee
• huncozyboy

시

🐰 생일을 축하하는 알림이 추가되었네요!
필드 하나, 매핑 하나, 간단하게 쏙!
알림 기능 확장되어 더욱 즐거워졌어요 🎂✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

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.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 주요 변경사항을 명확하게 설명합니다. '생일 알림 설정 조회 추가'는 PR의 핵심 변화를 정확히 반영합니다.
Description check	✅ Passed	PR 설명이 필수 템플릿 섹션을 포함하며, 작업 내용과 스크린샷으로 변경사항을 충분히 설명합니다.

✨ 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/LNK-32-Leenk-생일-알림-설정-조회-추가

---

📜 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 a60b56f and 4f7e21f.

📒 Files selected for processing (2)

• src/main/java/leets/leenk/domain/user/application/dto/response/NotificationSettingResponse.java (1 hunks)
• src/main/java/leets/leenk/domain/user/application/mapper/UserSettingMapper.java (1 hunks)

🔇 Additional comments (2)

src/main/java/leets/leenk/domain/user/application/dto/response/NotificationSettingResponse.java (1)

20-23: 새로운 필드가 올바르게 추가되었습니다.

isBirthdayNotify 필드가 기존 패턴을 잘 따르고 있으며, 다음 사항들이 적절히 처리되었습니다:

• @Schema 어노테이션을 통한 API 문서화
• 명확한 한글 설명 제공
• 다른 알림 설정 필드들과 일관된 네이밍 및 타입 사용
• 향후 필드 추가를 위한 trailing comma 적용

src/main/java/leets/leenk/domain/user/application/mapper/UserSettingMapper.java (1)

21-29: 매핑 로직 및 필드 정의 검증 완료 - DB 스키마 수동 확인 필요

코드 검증 결과:

• isBirthdayNotify 필드는 UserSetting 엔티티에 올바르게 정의됨 (nullable=false, default=true)
• NotificationSettingResponse DTO에 필드가 포함되어 있음
• 매퍼의 toNotificationSettingResponse() 메서드에서 모든 필드를 정확히 매핑함 ✓

다만, 저장소에 데이터베이스 마이그레이션 파일(SQL/XML)이 발견되지 않았습니다. 기존 user_setting 테이블에 is_birthday_notify 컬럼이 추가되었는지 다음 방법으로 확인하세요:

• 스프링 JPA/Hibernate DDL 자동 생성 여부 확인 (application.yml에서 spring.jpa.hibernate.ddl-auto 설정)
• 실제 데이터베이스에서 user_setting 테이블 컬럼 확인
• 마이그레이션 도구(Liquibase, Flyway 등) 사용 여부 확인 후 해당 마이그레이션 파일 배포 확인

---

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.}

[soo0711]

확인했습니다! 수고하셨습니다~ 👍