[Feature/#328] 알림 설정 API Request/Response 타입 정의

[jjjsun]

🚨 관련 이슈

Closed #328

✨ 변경사항

• 🐞 BugFix Something isn't working
• 💻 CrossBrowsing Browser compatibility
• 🌏 Deploy Deploy
• 🎨 Design Markup & styling
• 📃 Docs Documentation writing and editing (README.md, etc.)
• ✨ Feature Feature
• 🔨 Refactor Code refactoring
• ⚙️ Setting Development environment setup
• ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

• 알림 설정 API request/response 타입 정의
• Setting UI용 기존 타입은 페이지 로컬로 분리했습니다

😅 미완성 작업

N/A

📢 논의 사항 및 참고 사항

• 멤버 role은 TMemberRole 재사용

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

Summary by CodeRabbit

• 리팩터링
  • 알림 설정 관련 데이터 구조를 정리하고 관리 방식을 개선했습니다.
  • 알림 설정 조회 및 변경 요청을 일관된 형식으로 처리할 수 있도록 내부 구조를 재구성했습니다.
  • 이번 변경으로 화면 구성과 기존 알림 설정 사용 방식에는 변화가 없습니다.

[coderabbitai]

Warning

Review limit reached

@jjjsun, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 52 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: ab20fa4e-1849-4b7c-90ef-03f1909cd735

📥 Commits

Reviewing files that changed from the base of the PR and between 0808b24 and b99cf86.

📒 Files selected for processing (1)

• src/types/setting/notification.ts

📝 Walkthrough

Walkthrough

알림 설정 API의 조회·변경 요청 타입을 새로 정의하고, Setting 페이지의 기존 상태 타입을 로컬 인터페이스로 이동했습니다. 컴포넌트의 동작과 UI 렌더링 구조는 변경되지 않았습니다.

Changes

알림 설정 타입 정리

Layer / File(s)	Summary
Setting 로컬 상태 타입 정리 src/pages/setting/Setting.tsx	채널 및 워크스페이스 알림 상태 타입을 파일 내부에 정의하고 기본값 상수와 상태에 적용했습니다.
알림 설정 조회 계약 정의 src/types/setting/notification.ts	멤버 목록 조회와 내 알림 설정 조회를 위한 응답·파라미터 타입 및 공통 empty data 타입을 추가하고 기존 상태 타입을 제거했습니다.
알림 설정 변경 계약 정의 src/types/setting/notification.ts	조직, 멤버, 마스터, 채널, 알림 설정 PATCH 요청 바디 인터페이스를 추가했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• WhereYouAd/WhereYouAd-Frontend#216: Setting.tsx의 브라우저 푸시·이메일 알림 설정 상태와 타입을 함께 수정합니다.
• WhereYouAd/WhereYouAd-Frontend#322: 알림 설정 타입과 Setting.tsx의 상태 연결 부분을 함께 수정합니다.

Suggested reviewers: yermim, seojegyeong

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 알림 설정 API 타입 정의라는 핵심 변경을 정확히 요약합니다.
Description check	✅ Passed	필수 섹션인 관련 이슈, 변경사항, 작업 내용, 미완성 작업, 논의 사항이 모두 채워져 있어 템플릿을 대부분 충족합니다.
Linked Issues check	✅ Passed	직접 이슈 #328의 GET/PATCH 타입 정의와 Setting 로컬 타입 분리 요구를 모두 반영했습니다.
Out of Scope Changes check	✅ Passed	Setting UI용 타입 로컬 분리도 공통 타입 기반 정리 범위 안에 있어 별도 유출 변경은 보이지 않습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/#328

---

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

[github-actions]

📚 Storybook 배포 완료

항목	링크
📖 Storybook	https://69a147b60a56365d9e2185ef-nklvelydkx.chromatic.com/
🔍 Chromatic	https://www.chromatic.com/build?appId=69a147b60a56365d9e2185ef&number=457

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/types/setting/notification.ts`:
- Line 1: Update the import in the notification types module to use the
project’s `@/` alias instead of the relative path, while continuing to import
TMemberRole from the workspace types module.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 177b8ccd-a866-4ea4-a54d-37fb0d6144aa

📥 Commits

Reviewing files that changed from the base of the PR and between ffd9333 and 0808b24.

📒 Files selected for processing (2)

• src/pages/setting/Setting.tsx
• src/types/setting/notification.ts

[YermIm]

P4: 확인했습니다!