refactor: CameraAPIWorker 기존 Network 모듈 리펙토링 #705
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔵PR을 올리기 전 아래 사항을 확인해주세요.
(위 내용은 지워주세요)
😽개요
CameraAPIs
,CameraAPIWorker
를BBAPI
를 채택하여 리펙토링해요CameraAPIWorker
내부에 있는 인스턴스 메서드 네이밍을 수정 해요CameraAPIWorker
,CameraRepository
에 주석을 추가해요BBRxAPIWorker
내부Upload
통신 메서드를 구현해요Domain
,Data
모듈에 있는 Trash 폴더 일부를 제거해요🛠️작업 내용
APIWorker
를BBAPI
를 채택하여 리팩토링 작업했습니다. 추가로CameraAPIWorker
,CameraAPIs
에 주석 및 네이밍을 수정해서 어떤 API를 호출하는지 명확하게 구분할 수 있도록 하였습니다.BBRxAPIWorker
내부에 이미지 Upload를 위한upload
Method를 구현하였습니다. 그리고upload
같은 경우 서버에서 자체적으로 성공 여부를 넘겨 주지 않습니다. 그래서DataResponseSerializerProtocol
를 사용해서 응답 핸들러를 직접 커스텀으로 구현하였습니다.DataResponseSerializerProtocol
DataResponseSerializerProtocol
를 채택한BBUploadResponseSerializer
를 통해statusCode
가 200부터 300전까지 범위일 경우 true Return 하도록 하였습니다.✅테스트 케이스
🙏🏻아래와 같이 PR을 리뷰해주세요.