Skip to content

Conversation

@hsw1920
Copy link
Collaborator

@hsw1920 hsw1920 commented Nov 25, 2024

🤔 배경

EditPhotoFeatureDemo 개발 시 Guest와 Host의 커넥션연결을 위해 필요 없는 의존성을 가지고 있었습니다.
이를 분리하고자 DemoApp에서만 사용하는 SendOffer 를 요청하는 임시 뷰가 필요했습니다.

📃 작업 내역

  • 임시 뷰 SendOfferTempViewController 구현
  • EditPhotoRoomFeatureViewController에서 필요없는 SendOfferUseCase 제거

✅ 리뷰 노트

개발 환경 개선을 위한 작업이며 Debug에서만 활용하여 실제 release될 부분이 아니기 때문에 특별히 리뷰하실 부분은 없습니다.

  • EditPhotoFeatureDemo를 Run하면 SendOffer를 요청할 수 있습니다.
  • SceneDelegate에서 rootViewController를 Host/Guest로 나누어 실행할 때의 불편함을 개선하였습니다.

🎨 스크린샷

image

🚀 테스트 방법

EditPhotoFeatureDemoApp을 실행하면 됩니다.

hsw1920 and others added 3 commits November 25, 2024 13:49
- Demo 개발 시 Offer를 담당하는 뷰를 구현하였습니다.

Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
@hsw1920 hsw1920 added ✨ feat 새로운 기능 추가 ⚙️ refactor 코드 정상화 labels Nov 25, 2024
@hsw1920 hsw1920 requested a review from 0Hooni November 25, 2024 05:27
@hsw1920 hsw1920 self-assigned this Nov 25, 2024
@hsw1920 hsw1920 linked an issue Nov 25, 2024 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@0Hooni 0Hooni left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@hsw1920 hsw1920 merged commit f8fc8c9 into develop Nov 25, 2024
1 check passed
@hsw1920 hsw1920 deleted the feat/#96-sendofferusecase-viewcontroller branch November 25, 2024 05:29
@hsw1920 hsw1920 restored the feat/#96-sendofferusecase-viewcontroller branch November 25, 2024 06:50
@0Hooni 0Hooni deleted the feat/#96-sendofferusecase-viewcontroller branch November 27, 2024 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat 새로운 기능 추가 ⚙️ refactor 코드 정상화

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SendOfferUseCase 관리 ViewController 생성

3 participants