Skip to content

Comments

Hotfix/#134 빌드 에러 해결#137

Merged
y-eonee merged 1 commit intodevelopfrom
refactor/#134-자체-qa
Jan 21, 2026

Hidden character warning

The head ref may contain hidden characters: "refactor/#134-\uc790\uccb4-qa"
Merged

Hotfix/#134 빌드 에러 해결#137
y-eonee merged 1 commit intodevelopfrom
refactor/#134-자체-qa

Conversation

@y-eonee
Copy link
Contributor

@y-eonee y-eonee commented Jan 21, 2026

🔗 연결된 이슈

  • Closed: #이슈번호

📄 작업 내용

  • 어쩌구저쩌구
구현 내용 IPhone 16 pro IPhone 13 mini
GIF

✅ Testing

  • 테스트 목적과 상황

  • 시나리오 진행에 필요한 값

  • 시나리오 진행에 필요한 조건

  • 시나리오 완료 시 보장하는 결과

💻 주요 코드 설명

코드 설명할 파일 이름 (ex: HomeView)

  • 어쩌구저쩌구
// 코드는 이 사이에 작성하면 됩니다. 

📚 참고자료

👀 기타 더 이야기해볼 점

@y-eonee y-eonee requested a review from a team January 21, 2026 12:47
@y-eonee y-eonee self-assigned this Jan 21, 2026
@y-eonee y-eonee requested review from sum130 and wotjs020708 and removed request for a team January 21, 2026 12:47
@y-eonee y-eonee requested a review from soseoyo12 January 21, 2026 12:47
@y-eonee y-eonee added the Hotfix 충돌해결 등 devlop에 바로 반영해야하는 경우 사용 label Jan 21, 2026
@y-eonee y-eonee linked an issue Jan 21, 2026 that may be closed by this pull request
1 task
@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

개요

엔티티 타입을 ChallengeEntity/RoutineEntity에서 ProgressChallengeEntity/ProgressRoutineEntity로 이름 변경하고, ChallengeDemoAPI 엔드포인트를 DemoAPI로 통합하며, createChallenge 메서드를 ChallengeRepository에서 DemoRepository로 이동하고 관련 의존성을 업데이트합니다.

변경 사항

집합 / 파일(들) 변경 요약
DTO 및 응답 모델
Cherrish-iOS/Cherrish-iOS/Data/Model/Demo/FetchChallengesResponseDTO.swift, Cherrish-iOS/Cherrish-iOS/Data/Model/RoutineToggleResponseDTO.swift
toEntity() 메서드의 반환 타입을 ChallengeEntityProgressChallengeEntity, RoutineEntityProgressRoutineEntity로 변경
API 엔드포인트
Cherrish-iOS/Cherrish-iOS/Data/Network/EndPoint/ChallengeDemoAPI.swift (삭제), Cherrish-iOS/Cherrish-iOS/Data/Network/EndPoint/DemoAPI.swift
ChallengeDemoAPI.swift 파일 제거; DemoAPIcreateChallenge(userID: Int, requestDTO:) 케이스 추가 및 라우팅 로직 통합
저장소
Cherrish-iOS/Cherrish-iOS/Data/Repository/ChallengeRepository.swift, Cherrish-iOS/Cherrish-iOS/Data/Repository/DemoRepository.swift
ChallengeRepository에서 createChallenge() 메서드 제거; DemoRepository에 추가 및 반환 타입을 새로운 엔티티 타입으로 변경
도메인 인터페이스
Cherrish-iOS/Cherrish-iOS/Domain/Interface/ChallengeInterface.swift, Cherrish-iOS/Cherrish-iOS/Domain/Interface/DemoInterface.swift
ChallengeInterface에서 createChallenge() 제거; DemoInterface에서 메서드 반환 타입 및 createChallenge() 추가
도메인 엔티티
Cherrish-iOS/Cherrish-iOS/Domain/Model/Demo/ProgressChallengeEntity.swift, Cherrish-iOS/Cherrish-iOS/Domain/Model/Demo/ProgressRoutineEntity.swift
엔티티 구조체 이름 변경: ChallengeEntityProgressChallengeEntity, RoutineEntityProgressRoutineEntity; 속성 타입 업데이트
유스케이스
Cherrish-iOS/Cherrish-iOS/Domain/UseCase/CreateChallengeUseCase.swift, Cherrish-iOS/Cherrish-iOS/Domain/UseCase/FetchChallengeUseCase.swift, Cherrish-iOS/Cherrish-iOS/Domain/UseCase/AdvanceDayUseCase.swift, Cherrish-iOS/Cherrish-iOS/Domain/UseCase/ToggleRoutineUseCase.swift
반환 타입을 새로운 Progress*Entity 타입으로 변경; CreateChallengeUseCase의 저장소 의존성을 ChallengeInterfaceDemoInterface로 변경
의존성 조립
Cherrish-iOS/Cherrish-iOS/Domain/DomainDependencyAssembler.swift
CreateChallengeUseCase 등록 시 challengeRepositorydemoRepository 사용으로 변경
뷰모델
Cherrish-iOS/Cherrish-iOS/Presentation/Feature/ChallengeView/Coordinator/ViewModel/ChallengeProgressViewModel.swift
@Published 속성의 타입을 ChallengeEntity?ProgressChallengeEntity?, [RoutineEntity][ProgressRoutineEntity]로 변경 및 인스턴스 생성 코드 업데이트

예상 코드 리뷰 노력

🎯 3 (중간) | ⏱️ ~20분

관련된 가능성 있는 PR

  • Hotfix/#132 챌린지 api 연동 #133: 데모 챌린지 생성 엔드포인트 관련 — 이전 PR이 ChallengeDemoAPIcreateChallenge 지원을 추가했다면, 현재 PR은 이를 DemoAPI로 통합하고 저장소/유스케이스를 조정
  • Feat/#121 챌린지 api 연결 #127: 동일한 데모 API/DTO/저장소/유스케이스 코드 경로 수정 — 이전 PR의 데모 API 통합 위에 엔티티 이름 변경 및 createChallenge 처리 추가

제안되는 리뷰어

  • wotjs020708
  • sum130
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/#134-자체-qa

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.

Copy link
Contributor

@soseoyo12 soseoyo12 left a comment

Choose a reason for hiding this comment

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

수고하셨슴당

Copy link
Contributor

@wotjs020708 wotjs020708 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.!

Copy link
Contributor

@sum130 sum130 left a comment

Choose a reason for hiding this comment

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

수정 감사합니당 어푸푸푸~~~

@y-eonee y-eonee merged commit 3da914b into develop Jan 21, 2026
1 check was pending
@y-eonee y-eonee deleted the refactor/#134-자체-qa branch January 21, 2026 12:49
This was referenced Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Hotfix 충돌해결 등 devlop에 바로 반영해야하는 경우 사용 나연🐹

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] 자체 QA

4 participants