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: 음성 댓글 리스트 추가 된 필드 값 반영해요 #758

Merged
merged 10 commits into from
Mar 5, 2025
Merged

fix: 음성 댓글 리스트 추가 된 필드 값 반영해요 #758

merged 10 commits into from
Mar 5, 2025

Conversation

Do-hyun-Kim
Copy link
Collaborator

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

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

😽음성 댓글 리스트 내부 추가 된 필드 값 반영해요

  • 음성 댓글 리스트 내부 voiceUrl 필드 값 추가된 부분 반영했습니다.
  • 음성 댓글 리스트 내에 comment는 옵셔널로 받기 때문에 옵셔널 처리하였습니다.

✅테스트 케이스

  • 음성 댓글 등록이 잘 되는지 확인해요
  • 음성 댓글 Caching이 되는지 확인해요

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

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

@Do-hyun-Kim
feat: DailyCalendarViewController FireLottieView 추가
196ca0e 
- swift.yml refactor 브런치 workflow 실행하도록 수정 및 if 구문 제거
- Fastlane destination iOS 17.0 으로 설정
@Do-hyun-Kim
feat: Fastlane build_app Timeout 시간 및 retry 횟수 변경
eb35d91 
- build_app xcargs 값 추가
@Do-hyun-Kim
feat: CI 환경 내에 사용 가능한 시뮬레이터 장치 출력 구문 추가
3856de6
@Do-hyun-Kim
fix: CI 환경에 설치되어 있는 시뮬레이터로 Build 하도록 수정
813f1f0
@Do-hyun-Kim
fix: Xcode Version 16.2로 변경
e7506fc
@Do-hyun-Kim
fix: generic destination으로 build 하도록 수정
d6afe70
@Do-hyun-Kim
fix: runner mac-os15 로 변경
d4fa67b
@Do-hyun-Kim
fix: Tuist AppVersion 1.3.0 으로 수정
9d346ff 
-  AppKey 1.3.0 버전으로 변경
@Do-hyun-Kim
feat: 음성 댓글 생성, 조회 API 내부 voiceURL 필드 값 추가
20df177 
- comment 필드 값 옵셔널 처리 추가
@Do-hyun-Kim
chore: github Action STG 배포 파이프 라인 수정
9d7b357
@Do-hyun-Kim Do-hyun-Kim requested a review from akrudal March 5, 2025 11:40
@Do-hyun-Kim Do-hyun-Kim self-assigned this Mar 5, 2025
@Do-hyun-Kim Do-hyun-Kim merged commit c162972 into develop Mar 5, 2025
@Do-hyun-Kim Do-hyun-Kim deleted the refactor/#756-add-voice-response-field branch March 5, 2025 13:53
akrudal added a commit that referenced this pull request Mar 11, 2025
@akrudal @Do-hyun-Kim
1.3.0 기능을 추가합니다. (#761)
5b7adff 
* Revert "fix: Core Module Simulator Build시 Core Module에 의존된 Firebase Crashlytics Crash issue 발생 해결 (#733)" (#746)

This reverts commit 558ff15.

* feat: 음성 댓글 API Worker를 추가해요  (#742)

* feat: 음성 댓글 관련 APIs, APIWorker 추가
- 음성 댓글 관련 Entities 추가

* feat: VoiceRepository, VoiceRepositoryProtocol 추가 및 내부 메서드 구현

* feat: Equalizer View를 구현해요 (#747)

* feat: BBRecorderCore, BBRecorderManger, BBRecorderOption AVAudioRecorder, AVAudioPlayer를 사용하기 위한 클래스들 추상화

* fix: mp4 FileName 수정
- AVRecorder 데시벨 데이터 생성 활성화 하도록 수정

* fix: BBRecorderManger RxControlEvent 추가
- AVAudioRecorderDelegateProxy 추가

* CommentTextFiledView RecordButton, Container 추가
- BBRecorderManger requestMicrophonePermission 옵저버블 Property 추가
- CommentViewController ViewController 진입 시 접근 권한 팝업 및 Record Session 초기 세팅 로직 추가

* feat: Tuist Project Info.plist 마이크 접근 권한 문구 추가

* feat: CommentTextFiledView 녹음 관련 로직 추가
- BBEqualizerView Wave 파형 및 dot UI 구현
- BBRecorderCore  DocumentsPath AVRecorder, AVPlayer와 불 일치 이슈 수정
- BBRecorderManger AVRecorder Decibles 가져오는 메서드 추가
- BBRecorder Reactive+Extensions Observable 타입 Decibles 가져오는 타입 추가
- 녹음 UI 관련 image Assets 추가

* feat: CommentTextFiledView CurrentTime에따라 ConfirmButton Enabled 처리 로직 추가
- CommentTextFiledView  BBEqualizerView Decibels Bind 처리 로직 추가
- CommentTextFiledView  CurrentTime Bind 처리 로직 추가
-  Reactive+Ext  requestCurrentTime 옵저버블 추가
-  requestRecordDecibels Method 제거 및 AVInputNode를 사용하여 실시간 오디오 데이터 처리 로직 추가

* feat: BBRecorderManager 블루투스 입력장치 허용 옵션 추가
- Reactive+Extensions 데시벨 요청 옵저버블 로직 수정
- CommentTextFieldView EqualizerView 바인딩 처리 로직 수정

* fix: Github Action Google Service Info  관련 실행 명령어 제거

* feat: 음성 댓글 비즈니스 로직 및 �CommentCell 스크롤 버벅임 현상 수정해요 (#749)

* feat: CreateVoiceCommentUseCase 비즈니스 로직 추가
- CommentDIContainer CreateVoiceCommentUseCase 의존성 주입

* feat: CommentDIContainer VoiceCommentPresignedURLUseCase 의존성 주입 코드 추가
- CommentTextFieldReactor 녹음 데이터 CommentViewReactor로 전달하는 코드 추가
- CommentViewReactor 음성 댓글 업로드, PresignedURL 발급, 추가 비즈니스 로직 코드 추가
- CommentService Global State 코드 추가
- PostCommentResponseDTO CommentType 필드 값 추가
- VoiceAPIWorker MediaFile 업로드 메서드 추가
- VoiceCommentPresignedURLUseCase 비즈니스 로직 코드 추가

* feat: BBDiskCacheStroageError Type 정의 및 ErrorDescription 문구 선언
- BBDiskCacheStroage Class 내부 기능 구현
- DesignSystem 녹음 기능 관련 image Assets 추가

* fix: CommentTextFieldReactor, CommentTextFieldView 취소 버튼 클릭 시 음성 댓글 추가하는 이슈 수정
- CommentViewReactor mp4 확장자명 m4a로 변경

* fix: BBEqualizerView BackgroundColor 설정 코드 추가
- BBRecorderCore m4a 확장자로 수정

* fix: CommentTextFieldView 이벤트 중복 발생 이슈 수정
- CommentTextFieldViewReactor 로직 수정
- CommentRepository 음성 녹음 URL FileManager 저장 로직 추가
- VoiceRepository 음성 녹음 API 생성 시 FileManager 저장 로직 추가

* fix: BBRecorderManager updateDecibels, normalizeDecibel 메서드 제거
- AVAudioPCMBuffer Extensions 파일 추가 및 Decibels 정규화 메서드 추가
- Reactive+Ext requestAudioFileDecibels Observable Type 추가

* fix: BBEqualizerState inital, record case로 분류
- BBEqualizerConfig inital State에 따라 UI 구성
- BBEqualizerView BBEqualizerState 상태 값에 따라 context Color, width, Point Y, X 값 설정
- BBRecorderCore, Manager AVAudioPlayer 관련 인스턴스 및 메서드 제거
- Reactive+Ext requestAudioCurrentTime ObservableType 추가

* feat: CommentCell 내부 BBEqualizerView Spacing 값 수정
- BBEqualizerView 상태 값에 따라 UIImage 변경 코드 추가
- Assets Catalog 일시정지 이미지 리소스 추가

* feat: CommentCellReactor setPlayAudioId Mutation 추가
- CommentCell playerManager Property 추가 및 재생 관련 기능 추가
- Reactive+Ext AVError를 통한 오류 처리 핸들링 로직 추가
- BBRecorderManager 재생 관련 메서드 추가

* fix: BBDiskCacheStorage async Method로 수정
- CommentRepository, VoiceRepository Blocking 발생 코드 수정

* fix: VoiceRepository Blocking 발생 코드 수정

* fix: CommentCellReactor equalizerState Pulse 로 변경
- CommentCell 재생 및 중지 관련 메서드 기능 추가
- CommentCell 재생 시 남은 시간 Binding 로직 추가
- BBEqualizerView deinit 구문 추가
- Reactive+Ext willChangedAudioTime ObservableType 추가

* fix: BBEqualizerView prepareForReuse 호출 시 invalid Layout Update 되는 이슈 수정
- CommentCellReactor comment Pulse로 변경
- Reactive+Ext willChangedAudioTime Observable inital Type 일때 방출 안되는 이슈 수정

* feat: CommentCellReactor prepareForReuse 호출 시 이전 설정된 UI State 값 초기화
- CommentCell prepareForReuse 메서드 내부 녹화 중일 시 AVAudioPlayer 중지 메서드 추가
- Reactive+Ext 로그 제거

* fix: BBUploadResponseSerializer 내부 강제 언래핑 제거 및 BBUploadError로 throw

* chore: debug 구문 제거

* fix: CommentCell 스크롤 버벅임 이슈 수정

* chore: CommentTextFieldReactor, CommentCell, CommentTextFieldView 출력 구문 제거 및 RxScheduler로 변경

* feat: 네트워크 통신 시 타임아웃이 일어나면 재요청 하도록 로직을 추가해요  (#754)

* fix: BBNetworkInterceptor Timeout 발생 시 Retry 할 수 있도록 로직 추가 RefreshToken Retry 코드 분기 처리

* fix: CreatePresignedURLUseCase 내부 책임 분리
- CreateImageUploadUseCase 이미지 업로드 로직 추가
- CameraDisplayViewReactor PresignedURL 및 Image Upload 완료 될때까지 로딩 인디케이터 띄우도록 로직 수정

* chore: BBNetworkInterceptor 로그용 출력구문 제거

* feat: 음성 댓글 삭제 API 및 오류 핸들링 로직을 추가해요 (#752)

* feat: 앱 리뷰 팝업 창을 띄우도록 기능을 추가해요  (#755)

* feat: MyDIContainer FetchUserCreateAtInfoUseCase 의존성 주입 코드 추가
- Tuist 음성 권한 팝업 문구 수정
- UserDefaultsKey 최신버전, 리뷰 카운트, 마지막 업데이트 날짜 저장 Key 등록
- MyRepository 최신버전, 리뷰 카운트, 마지막 업데이트 날짜 Update, Fetch 메서드 정의
- MyUserDefaults 내부 최신버전, 리뷰 카운트, 업데이트 날짜 저장 메서드 추가
- FetchUserCreateAtInfoUseCase 리뷰 팝업 정책에 따른 로직 추가

* fix: MainNavigator Alert 사라지지 않는 이슈 수정
- MainViewController 초기 NoEmptyView 안뜨는 이슈 수정

* feat: Extensions 폴더 내부 SKStoreReviewController+Ext 파일 및 리뷰 팝업 메서드 추가

* feat: MainViewReactor 별점 팝업 관련 State, UseCase 추가
- MainViewController isShowRatingAlert State에따라 팝업 띄우는 로직 추가
- ProfileViewReactor 테스트 코드 제거
- UserDefaultsKey isLatestVersion Key 추가, latestVersion 제거
- MyRepository fetchIsLatestVersion 메서드 추가, fetchLatestVersion 메서드 제거
- MeAPIWorker latest 필드 값 저장 로직 추가
- fetchUserCreateAtInfoUseCase 로직 수정

* fix: CameraDisplayViewReactor Survival 피드 업로드 이후 RatingUseCase 로직 실행 구문 추가
- MainViewControllerWrapper isRatingAlertHidden Flag 값 추가
- CameraDisplayNavigator isRatingHidden Parameter 추가
- AccountResingViewReactor FCMToken 제거 API 추가 및 Keychain Storage FCM Token 제거 로직 추가
- MeAPIWorker FCMToken Local Storage에 저장하는 로직 추가

* chore: github Action Push braches Event 실행 하도록 추가
- Xcode 16.2 환경으로 실행 하도록 수정

* fix: FetchUserCreatedAtInfoUseCase로 네이밍 수정
- MyRepository 내부 fetchFamilyCreateAt 메서드 네이밍 CreatedAt 메서드로 네이밍 수정
- MyDIContainer FetchUserCreatedAtInfoUseCaseProtocol 및 makeFetchUserCreatedAtInfoUseCase로 네이밍 수정

* 프로필 뷰와 알림 화면을 구현해요 (#757)

* feat: add profile view

* feat: add profileview

* feat: add notification view

* feat: add profile

* feat: add deeplink manager

* feat: add depp links

* feat: add review

* feat: add post deeplink

* feat: add notifi data

* feat: add review

* feat: remove file

* fix: 음성 댓글 리스트 추가 된 필드 값 반영해요 (#758)

* feat: DailyCalendarViewController FireLottieView 추가
- swift.yml refactor 브런치 workflow 실행하도록 수정 및 if 구문 제거
- Fastlane destination iOS 17.0 으로 설정

* feat: Fastlane build_app Timeout 시간 및 retry 횟수 변경
- build_app xcargs 값 추가

* feat: CI 환경 내에 사용 가능한 시뮬레이터 장치 출력 구문 추가

* fix: CI 환경에 설치되어 있는 시뮬레이터로 Build 하도록 수정

* fix: Xcode Version 16.2로 변경

* fix: generic destination으로 build 하도록 수정

* fix: runner mac-os15 로 변경

* fix: Tuist AppVersion 1.3.0 으로 수정
-  AppKey 1.3.0 버전으로 변경

* feat: 음성 댓글 생성, 조회 API 내부 voiceURL 필드 값 추가
- comment 필드 값 옵셔널 처리 추가

* chore: github Action STG 배포 파이프 라인 수정

* 네비게이션 바 이미지 스케일을 수정합니다

---------

Co-authored-by: 김도현 <[email protected]>
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
None yet
Projects
None yet
Milestone
No milestone
Development

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