Skip to content

[FEATURE] 외주 상세 조회시 디자이너 참여 여부 필드값 추가#110

Merged
fervovita merged 1 commit into
devfrom
feat/#109-commission-detail-add-field
Jul 9, 2026
Merged

[FEATURE] 외주 상세 조회시 디자이너 참여 여부 필드값 추가#110
fervovita merged 1 commit into
devfrom
feat/#109-commission-detail-add-field

Conversation

@fervovita

@fervovita fervovita commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

🚀 Related issue

Closes #109

#️⃣ Summary

  • 외주 상세 조회 시 디자이너에게만 지원 여부(applied)를 내려줘 프론트가 참여/참여취소 버튼을 분기할 수 있도록 했습니다.

🔧 Changes

  • CommissionDetailResponse에 디자이너 지원 여부 필드 applied 추가

📸 Test Evidence

  • 강사
image
  • 디자이너
image

💬 Reviewer Notes

  • applied는 priceInfo와 동일하게 디자이너 전용 값으로, 조회자의 역할/상태에 따라 3가지로 내려갑니다.
    • true -> RECRUITING & PENDING 지원 있음
    • false -> RECRUITING & 미지원
    • null -> 강사이거나 모집 중이 아님

Summary by CodeRabbit

  • 새 기능

    • 디자이너가 커미션 상세를 볼 때, 지원 여부를 함께 확인할 수 있습니다.
    • 디자이너 전용 가격 정보에 대한 안내 문구가 더 명확해졌습니다.
  • 개선 사항

    • 커미션 상세 응답에 디자이너 관점의 추가 상태 정보가 포함되도록 확장되었습니다.
    • 일부 가격 정보 표기가 더 구체적으로 다듬어졌습니다.

@fervovita fervovita self-assigned this Jul 8, 2026
@fervovita fervovita requested a review from Jong0128 as a code owner July 8, 2026 07:18
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: 1aff80d3-cac2-431e-b372-1e6255833a21

📥 Commits

Reviewing files that changed from the base of the PR and between 36c9c05 and 8a4e4d5.

📒 Files selected for processing (3)
  • src/main/java/ditda/backend/domain/commission/core/dto/response/CommissionDetailResponse.java
  • src/main/java/ditda/backend/domain/commission/core/facade/CommissionFacade.java
  • src/main/java/ditda/backend/domain/commission/core/mapper/CommissionDetailMapper.java

📝 Walkthrough

Walkthrough

외주 상세 조회 응답에 디자이너의 지원 여부를 나타내는 applied 필드가 추가되었다. CommissionFacade는 커미션 상태가 RECRUITING이고 조회자가 디자이너인 경우 ApplicationService를 통해 대기 중인 지원 여부를 계산하고, CommissionDetailMapper를 통해 응답에 전달한다.

Changes

외주 상세 조회 지원 여부 필드 추가

Layer / File(s) Summary
응답 DTO 필드 및 문서화
src/main/java/ditda/backend/domain/commission/core/dto/response/CommissionDetailResponse.java
applied(Boolean) 필드가 추가되고 priceInfo, PriceInfo의 Schema 설명이 디자이너 관점으로 수정됨.
매퍼 시그니처 확장
src/main/java/ditda/backend/domain/commission/core/mapper/CommissionDetailMapper.java
toResponseBoolean applied 파라미터를 추가로 받아 응답 생성자에 전달하도록 변경됨.
Facade의 지원 여부 계산 및 전달
src/main/java/ditda/backend/domain/commission/core/facade/CommissionFacade.java
ApplicationService 의존성이 추가되고, 상태가 RECRUITING이며 디자이너인 경우에만 existsPendingApplication 결과를 applied로 계산해 toResponse 호출에 전달함.

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

Possibly related PRs

  • Ditda-Official/Ditda-Backend#22: CommissionApplication/ApplicationStatus 도메인과 레포지토리 계층을 추가하여 본 PR의 existsPendingApplication(...) 호출이 의존하는 기반을 제공함.
  • Ditda-Official/Ditda-Backend#65: 같은 CommissionDetailResponse/CommissionFacade/CommissionDetailMapper의 가격 응답 조립 로직을 수정·확장함.
  • Ditda-Official/Ditda-Backend#95: 본 PR에서 사용하는 existsPendingApplication 관련 pending 지원 조회/존재 판별 로직을 구현함.

Suggested reviewers: Jong0128

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목이 외주 상세 조회에서 디자이너 참여 여부 필드 추가라는 핵심 변경을 정확히 요약합니다.
Linked Issues check ✅ Passed 응답에 디자이너 참여 여부를 나타내는 applied 필드를 추가해 참여/취소 분기 목적을 충족합니다.
Out of Scope Changes check ✅ Passed 요구사항과 무관한 변경은 보이지 않으며, 응답 DTO·매퍼·페이사이드 로직만 수정되었습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/#109-commission-detail-add-field

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.

@Jong0128 Jong0128 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.

수고하셨습니다 😄
바로 merge해도 좋을꺼같습니다!!

@fervovita fervovita merged commit 31c7679 into dev Jul 9, 2026
2 checks passed
@fervovita fervovita deleted the feat/#109-commission-detail-add-field branch July 9, 2026 10:13
@fervovita fervovita mentioned this pull request Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 외주 상세 조회시 디자이너 참여 여부 필드값 추가

2 participants