Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: Camera 관련 비즈니스 로직 부분 리팩토링 해요 #718

Merged
merged 1 commit into from
Dec 10, 2024
Merged

fix: Camera 관련 비즈니스 로직 부분 리팩토링 해요 #718

merged 1 commit into from
Dec 10, 2024

Conversation

Do-hyun-Kim
Copy link
Collaborator

🔵PR을 올리기 전 아래 사항을 확인해주세요.

  • 구현한 로직과 기능이 올바르게 작동되는지 충분히 테스트해주세요.
  • 코드의 성능이나 메모리 효율성이 적절하게 고려되었는지, 불필요한 코드가 없는지 검토해주세요.
  • 이번 PR에서 구현된 주요 기능이나 해결된 문제에 대해 자세히 서술해주세요.
    (위 내용은 지워주세요)

😽개요

  • Camera API Method 호출 실패 시 오류 팝업 띄우도록 로직을 추가했습니다.
  • CameraRepositoryProtocol에 S3Bucket에 Upload 하는 Method를 추가했습니다.

🛠️작업 내용

  • CameraRepositoryProtocol에 S3Bucket Upload 하는 Method 추가
  • `BBAlert에 업로드 실패 관련 UI 추가
  • CameraNavigator, CameraDisplayNavigator 내부 메서드에 BBAlert 호출 메서드 추가
  • RealEmojiUseCase 코드 리팩토링
업로드 실패 시 화면

✅테스트 케이스

  • 카메라 관련 API 호출이 잘 되는지 확인해요
  • 카메라 API 호출 실패 시 오류 팝업이 뜨는지 확인해요

🙏🏻아래와 같이 PR을 리뷰해주세요.

  • PR 내용이 부족하다면 보충 요청해주세요.
  • 코드 스타일이 팀의 규칙에 맞게 작성되었는지, 일관성을 유지하고 있는지 확인해주세요.
  • 코드에 대한 문서화나 주석이 필요한 부분에 적절하게 작성되어 있는지 확인해주세요.
  • 구현된 로직이 효율적이고 올바르게 작성되었는지, 아키텍처를 잘 준수하고 있는지 검토해주세요.
  • 네이밍, 포매팅, 주석 등 코드의 일관성이 유지되고 있는지 확인해주세요.

@Do-hyun-Kim
feat: BBAlert secondaryAction Parameters 추가
c464266 
- BBAlert uploadFailed AlertView UI 추가
- CameraNavigator, CameraDisplayNavigator errorAlert Method 추가
- CameraRepository uploadEmoji Method 추가
@Do-hyun-Kim Do-hyun-Kim requested a review from akrudal December 9, 2024 08:19
@Do-hyun-Kim Do-hyun-Kim self-assigned this Dec 9, 2024
akrudal
akrudal approved these changes Dec 9, 2024
View reviewed changes
Copy link
Collaborator

@akrudal akrudal left a comment

Choose a reason for hiding this comment

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

이야.. 카메라.. 이제 .. 고슈인데여?

@akrudal
Copy link
Collaborator

akrudal commented Dec 9, 2024

확인했습니다 굳굳

@Do-hyun-Kim
Copy link
Collaborator Author

이야.. 카메라.. 이제 .. 고슈인데여?

ㅋㅋㅋㅅㅋ 경미님은 녹음 마스터 하러 ㄱㄱ

@Do-hyun-Kim Do-hyun-Kim merged commit b3352e0 into develop Dec 10, 2024
@Do-hyun-Kim Do-hyun-Kim deleted the refactor/#717-camerea-business-logic-modify branch December 10, 2024 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akrudal akrudal akrudal approved these changes

Assignees

@Do-hyun-Kim Do-hyun-Kim

Labels
✨ FEATURE 기능 개발 🦊 도현
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Do-hyun-Kim @akrudal