Skip to content

[Feat/#85] 학부 응답을 소프트웨어학부·인공지능학부로 표시 - #90

Open
tnals0924 wants to merge 1 commit into
feat/#84-member-oauth-profilefrom
feat/#85-department-display-name
Open

tnals0924 wants to merge 1 commit into
feat/#84-member-oauth-profilefrom
feat/#85-department-display-name

Conversation

@tnals0924

@tnals0924 tnals0924 commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

#️⃣연관된 이슈

🎯 해결하려는 문제가 무엇인가요?

학부를 응답으로 내려줄 때 enum 이름(SW/AI)이 그대로 나갑니다.

❓ 왜 해결해야 하나요?

화면에는 "소프트웨어학부"·"인공지능학부"로 보여야 합니다. 판별은 소속 문자열의 전공명("소프트웨어전공"·"인공지능전공")으로 하지만, 표시는 학부명으로 합니다.

⭐ 어떻게 해결했나요?

  • Department에 displayName 추가
  • 운영진 회비 조회 응답(AdminFeeSearchResponse)의 department를 name() 대신 displayName()으로 변경

🧩 이 PR의 한계 & 트레이드오프

없습니다.

⛓️ 기존 기능에 미치는 영향

  • 운영진 회비 조회 API의 department 값이 SW/AI → 소프트웨어학부/인공지능학부로 바뀝니다. 운영진 웹이 이 값으로 분기하고 있다면 함께 수정해야 합니다.

🔀 Edge Case & 실패 시나리오

  • 탈퇴한 회원의 과거 요청은 기존처럼 department가 null입니다.

📋 검토한 대안과 선택 이유

  • 클라이언트에서 코드→학부명 매핑: 학생 앱·운영진 웹마다 매핑이 흩어져 서버에서 내려주기로 했습니다.

💬 리뷰 포인트

  • 운영진 웹 영향 확인

@tnals0924 tnals0924 self-assigned this Sep 30, 2026
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • main

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: billilge/stream-server/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 8724fa50-33a3-416b-8b82-ec367ecbf7cc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@jjunh33 jjunh33 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

@xeoxxn xeoxxn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인 완료했습니다~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants