쿠링 2.5.1 버전

[hwan-cs]

작업 내용

• 동아리 목록 DTO(Club)에 RecruitmentStatus 필드 추가
  • 기존에 없어서 recruitmentStartDate, recruitmentEndDate로 날짜 계산 로직으로 지원 마감 기준을 파악했었음
• 공지사항 알림 DTO에 id 필드를 String으로 파싱하도록 수정. String으로 먼저 파싱 후, Int 변환할 것
  • 서버에서 int 형식으로 보내면 안되는 이슈가 있다고 함
  • 이제 진짜 공지사항 알림 탭 동작이 되어야하는게 맞을 듯..
• 동아리 이미지 aspectRatio 변경

스크린샷

AS-IS	TO-BE

Summary by CodeRabbit

• New Features

  • 클럽 카드와 상세 화면에서 모집 상태가 더 정확하게 표시됩니다.
  • 모집 마감일을 확인할 수 없을 때도 상태 문구가 표시됩니다.

• Bug Fixes

  • 클럽 목록 조회가 항상 기본 전체 조건으로 동작하도록 조정되었습니다.
  • 알림/공지 데이터의 식별자 처리 안정성이 개선되었습니다.

• Style

  • 일부 화면 구성과 표시 정렬이 정리되었습니다.

[coderabbitai]

Caution

Review failed

An error occurred during the review process. Please try again later.

Walkthrough

Club에 모집 상태가 추가되고, 클럽 목록 조회와 카드·상세 화면이 이를 기준으로 모집 문구를 표시하도록 바뀌었습니다. Notice는 userInfo의 id를 문자열에서 정수로 변환하도록 수정됐습니다.

Changes

클럽 모집 상태 흐름

Layer / File(s)	Summary
모델 계약 Shared/Models/Sources/Clubs.swift	Club에 recruitmentStatus가 추가되고, 공개 이니셜라이저와 mock이 이 값을 채우도록 바뀌었습니다.
목록 조회와 카드 표시 Features/ClubsFeatures/Sources/ClubsList.swift, UIKit/ClubsUI/Sources/ClubsList/Views/ClubCardView.swift	클럽 목록 조회가 (.all, [])로 고정되고, 카드의 모집 여부, 이미지 비율, 요약 색, 모집 종료 문구가 recruitmentStatus 기준으로 바뀌었습니다.
상세 문구 UIKit/ClubsUI/Sources/ClubsDetail/ClubInfoDetailView.swift	날짜 파싱 실패 시 상세 화면의 D-day 문구가 club.recruitmentStatus.rawValue로 바뀌었습니다.

공지 id 파싱

Layer / File(s)	Summary
id 변환 Shared/Models/Sources/Notice.swift	Notice(userInfo:)가 id를 문자열로 읽은 뒤 정수로 변환하도록 바뀌었습니다.

Possibly related PRs

• ku-ring/ios-app#285: ClubCardView와 같은 클럽 목록 UI를 추가한 PR로, 이번 변경의 카드 표시 로직과 직접 맞닿아 있습니다.
• ku-ring/ios-app#286: ClubInfoDetailView의 D-day 표시 경로를 다루며, 이번 PR의 상세 문구 변경과 같은 화면 흐름입니다.
• ku-ring/ios-app#289: getClubsList API 파라미터를 다루는 PR로, 이번 PR의 목록 조회 인자 변경과 연결됩니다.

Suggested labels

새기능/개선작업

Poem

🐰 깡총깡총, 클럽 표정이 바뀌었어요
모집 상태가 카드와 상세에 또렷해졌고
공지의 숫자도 문자열에서 톡, 정수로
토끼 발자국처럼 흐름이 가지런해졌답니다
당근 한 입, 리뷰 한 점!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	제목이 버전명만 담고 있어 이번 변경의 핵심 내용을 전달하지 못합니다.	버전명 대신 동아리 모집 상태 추가 등 핵심 변경사항을 한 문장으로 요약해 주세요.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description check	✅ Passed	작업 내용과 스크린샷 섹션이 포함되어 있어 템플릿의 핵심 항목은 대부분 충족합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/improve_club_1

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[hwan-cs]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.