Skip to content

[FIX#324] id=>name으로 수정#325

Merged
HongGeonUi merged 1 commit into
mainfrom
fix#324
Jan 27, 2026
Merged

[FIX#324] id=>name으로 수정#325
HongGeonUi merged 1 commit into
mainfrom
fix#324

Conversation

@HongGeonUi

@HongGeonUi HongGeonUi commented Jan 27, 2026

Copy link
Copy Markdown
Collaborator

📝 작업 내용

  • id에서 name으로 수정

🎯 관련 이슈

Closes #324

🔄 변경사항

👀 리뷰 포인트

📱 스크린샷/영상

Before

After

Summary by CodeRabbit

릴리스 노트

  • Refactor
    • 프로그램 데이터 처리 로직을 개선했습니다.

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

@HongGeonUi HongGeonUi self-assigned this Jan 27, 2026
@HongGeonUi HongGeonUi added ♻️ refactor 리팩토링 BE 백엔드 labels 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:42pm

@coderabbitai

coderabbitai Bot commented Jan 27, 2026

Copy link
Copy Markdown

Walkthrough

formatProgramData 함수에서 리더 사용자 ID를 추출하는 로직을 수정했어. 첫 번째 롤업 항목의 plain_text 필드를 사용하던 걸 name 필드로 변경했고, 다른 로직이나 제어 흐름은 건들지 않았어.

Changes

Cohort / File(s) 변경 내용
리더 사용자 ID 필드 변경
be/functions/src/services/programService.js
formatProgramData 함수에서 리더 사용자 ID 추출 시 rollup 값의 plain_text 필드 대신 name 필드 사용으로 변경

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

🐛 bug

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 실제 변경사항과 일치하며, 문제 #324를 수정하기 위해 id에서 name으로의 변경을 명확하게 나타낸다.
Description check ✅ Passed 기본 템플릿을 따르고 관련 이슈를 명시했으나, 변경사항(🔄) 섹션과 리뷰 포인트(👀) 섹션이 불완전하게 작성되었다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 이슈 #324의 요구사항 'id => name으로 수정'을 충족한다. formatProgramData에서 plain_text 대신 name 필드를 사용하도록 변경했다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 이슈 #324의 범위 내에 있으며, 리더 사용자 파라미터를 id에서 name으로 수정하는 목표에 집중되어 있다.
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: db33a25
  • Branch: fix#324
  • 빌드 완료 시간: 2026. 1. 27. 오후 10:42:08

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

@HongGeonUi
HongGeonUi merged commit fea05b7 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 백엔드 ♻️ refactor 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

리더사용자 파라미터 수정

1 participant