Skip to content

feat: 랭킹 관리자 제외 + 프로필 페이지 UX 개선#68

Merged
bbbang105 merged 2 commits intodevfrom
feat/exclude-admins-from-ranking
Mar 25, 2026
Merged

feat: 랭킹 관리자 제외 + 프로필 페이지 UX 개선#68
bbbang105 merged 2 commits intodevfrom
feat/exclude-admins-from-ranking

Conversation

@bbbang105
Copy link
Owner

Summary

  • 웹/봇 랭킹에서 관리자 및 지정 멤버 제외 (config 테이블 기반)
  • 프로필 수정 버튼 위치 개선 + 푸시 알림 유도 말풍선 추가

Changes

랭킹 필터링

파일 변경
packages/web/src/app/api/ranking/route.ts 관리자 + ranking_excluded_ids config 기반 필터링 추가
packages/bot/src/schedulers/weekly-ranking.ts env ADMIN_DISCORD_IDS + config ranking_excluded_ids 기반 필터링 추가

프로필 UX 개선

파일 변경
packages/web/src/app/(user)/profile/page.tsx 프로필 수정 버튼 헤더 우측 이동, 푸시 알림 유도 말풍선 추가

Design Decisions

결정 이유
config 테이블(ranking_excluded_ids) 사용 하드코딩 없이 DB에서 제외 대상 관리, 코드 변경/배포 불필요
관리자 제외는 기존 ADMIN_DISCORD_IDS + admin_discord_ids config 활용 이미 존재하는 인프라 재사용
푸시 유도를 Notification.permission 기반으로 체크 기기별 독립 동작, 불필요한 DB 호출 없음
알림 켜기 성공 시 상세 설정 페이지 이동 타입별 알림 세부 설정 유도

Test Plan

  • 웹 랭킹 페이지에서 관리자 멤버가 표시되지 않는지 확인
  • config ranking_excluded_ids에 등록된 멤버가 랭킹에서 제외되는지 확인
  • 봇 주간 랭킹 Discord 메시지에서 관리자/제외 멤버 미표시 확인
  • 프로필 페이지에서 "프로필 수정" 버튼이 헤더 우측에 위치하는지 확인
  • 푸시 알림 미허용 브라우저에서 말풍선이 표시되는지 확인
  • "알림 켜기" 버튼 클릭 → 권한 허용 → 알림 설정 페이지 이동 확인
  • 이미 알림 허용된 브라우저에서 말풍선이 표시되지 않는지 확인

🤖 Generated with Claude Code

bbbang105 and others added 2 commits March 25, 2026 22:55
- 웹 랭킹 API: 관리자 Discord ID + config `ranking_excluded_ids` 기반 필터링
- 봇 주간 랭킹: env `ADMIN_DISCORD_IDS` + config `ranking_excluded_ids` 기반 필터링
- config 테이블로 제외 대상 관리 (코드 변경 없이 DB에서 추가/제거 가능)

Co-Authored-By: Claude <noreply@anthropic.com>
- 프로필 수정 버튼을 헤더 우측으로 이동 (하단 → 상단)
- 푸시 알림 미허용 시 유도 말풍선 표시 (벨 펄스 애니메이션 + 알림 켜기 CTA)
- 알림 켜기 성공 시 상세 설정 페이지로 자동 이동

Co-Authored-By: Claude <noreply@anthropic.com>
@bbbang105 bbbang105 requested a review from choihooo as a code owner March 25, 2026 14:19
@bbbang105 bbbang105 added the 🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 label Mar 25, 2026
@vercel
Copy link

vercel bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
study-admin-web Ready Ready Preview, Comment Mar 25, 2026 2:20pm

@bbbang105 bbbang105 merged commit 60f914b into dev Mar 25, 2026
8 checks passed
@bbbang105 bbbang105 deleted the feat/exclude-admins-from-ranking branch March 25, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant