Skip to content

[FIX#320] TMI 플젝 관련 알림 작성자명 이슈#323

Merged
7ijin01 merged 4 commits into
mainfrom
FIX#320
Jan 27, 2026
Merged

[FIX#320] TMI 플젝 관련 알림 작성자명 이슈#323
7ijin01 merged 4 commits into
mainfrom
FIX#320

Conversation

@7ijin01

@7ijin01 7ijin01 commented Jan 27, 2026

Copy link
Copy Markdown
Collaborator

📝 작업 내용

  • tmi 플젝 관련 알림을 실명 기준으로 동작하도록 알림 작성자명을 수정했습니다

🎯 관련 이슈

Closes #320

Summary by CodeRabbit

버그 수정

  • 좋아요 알림에서 사용자 이름이 올바르게 표시되도록 개선했습니다.
  • 프로그램 유형에 따라 닉네임과 실명 표시 로직을 최적화했습니다.
  • 커뮤니티 회원 정보를 활용하여 더 정확한 사용자 이름 확인 기능을 추가했습니다.

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

@7ijin01 7ijin01 self-assigned this Jan 27, 2026
@7ijin01 7ijin01 added the 🐛 bug 버그 label Jan 27, 2026
@7ijin01 7ijin01 linked an issue Jan 27, 2026 that may be closed by this pull request
1 task
@7ijin01 7ijin01 added the BE 백엔드 label Jan 27, 2026
@vercel

vercel Bot commented Jan 27, 2026

Copy link
Copy Markdown

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

Project Deployment Review Updated (UTC)
youthit Ready Ready Preview, Comment Jan 27, 2026 1:23pm

@coderabbitai

coderabbitai Bot commented Jan 27, 2026

Copy link
Copy Markdown

Walkthrough

댓글, 좋아요 알림에서 작성자 표시명을 결정하는 로직을 변경했어. 기존의 isPublic 플래그 기반 처리에서 programType을 활용한 흐름으로 전환했고, TMI 프로젝트면 실명, 아니면 커뮤니티/글로벌 닉네임을 우선 사용하도록 통합했어.

Changes

Cohort / File(s) 변경사항
알림 시스템의 작성자명 해석 로직 개선
be/functions/src/services/commentService.js, be/functions/src/services/communityService.js
programType 기반 분기 처리 추가: TMI 프로젝트면 users 컬렉션의 실명 직접 조회, 그 외는 community 멤버십 닉네임 → 글로벌 nicknames → users 이름 순으로 폴백. 기존 post.isPublic 기반 이원화 로직 제거하고 통합 경로로 변경

Sequence Diagram(s)

sequenceDiagram
    participant System as 알림 시스템
    participant Post as Post/Community<br/>Data
    participant User as Users<br/>Collection
    participant NickGlobal as Global<br/>Nicknames
    participant NickCom as Community<br/>Member Data

    rect rgba(255, 200, 100, 0.5)
    Note over System: 기존 흐름: isPublic 기반 분기
    System->>Post: post.isPublic 확인
    alt 공개 게시글
        System->>NickGlobal: 글로벌 닉네임 조회
        NickGlobal-->>System: 닉네임 반환
    else 비공개 게시글
        System->>User: 실명 조회
        User-->>System: 실명 반환
    end
    end

    rect rgba(100, 200, 255, 0.5)
    Note over System: 새 흐름: programType 기반 분기
    System->>Post: programType 확인
    alt TMI 프로젝트
        System->>User: 실명 조회
        User-->>System: 실명 반환
    else 일반 프로젝트
        System->>NickCom: 커뮤니티 멤버 닉네임 조회
        alt 멤버 닉네임 존재
            NickCom-->>System: 멤버 닉네임 반환
        else 멤버 닉네임 없음
            System->>NickGlobal: 글로벌 닉네임 조회
            alt 글로벌 닉네임 존재
                NickGlobal-->>System: 글로벌 닉네임 반환
            else 글로벌 닉네임 없음
                System->>User: 사용자 이름 조회
                User-->>System: 사용자 이름 반환
            end
        end
    end
    end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • [FIX#225] 푸쉬 알림 메시지 수정 #226: commentService.js와 communityService.js의 알림 메시지 구성 로직을 변경하는 PR으로, 동일 파일의 알림 관련 흐름을 수정하는 이번 PR과 함께 검토 필요함.
🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명에서 작업 내용과 관련 이슈를 명시했지만, 리뷰 포인트(👀 섹션)와 변경사항(🔄 섹션)이 채워지지 않아 필수 정보가 부족하다. 리뷰 포인트와 변경사항 섹션을 구체적으로 작성해서 코드 리뷰어가 쉽게 이해할 수 있도록 보완해 주세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 이슈 번호를 명시하고 TMI 프로젝트의 알림 작성자명 문제를 정확히 요약하고 있어서, 주요 변경사항과 잘 맞아떨어진다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 이슈 #320의 요구사항인 TMI 프로젝트의 게시글/댓글 좋아요 알림 작성자명을 실명으로 처리하도록 수정된 것과 일치한다.
Out of Scope Changes check ✅ Passed commentService.js와 communityService.js의 변경사항이 모두 알림 작성자명 해결 로직 개선에만 집중되어 있고, 이슈 범위를 벗어나는 변경은 없다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

@github-actions

Copy link
Copy Markdown

🔧 빌드 상태: 성공! ✅

Firebase Functions 빌드 및 에뮬레이터 기동이 성공적으로 완료되었습니다! 🎉

✅ 체크 완료

  • 🔧 Backend Build (Emulator): 성공

📊 상세 정보

  • Commit: a33d55a
  • Branch: FIX#320
  • 빌드 완료 시간: 2026. 1. 27. 오후 10:23:57

🚀 Functions 빌드 및 실행에 문제가 없습니다. 머지 가능합니다!

@7ijin01
7ijin01 merged commit a97d556 into main Jan 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BE 백엔드 🐛 bug 버그

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TMI 플젝 관련 알림 작성자명 이슈

1 participant