Refactor/#144 서버통신 로직 async await로 리팩토링 #146

Neph3779 · 2023-02-11T04:41:21Z

안녕하세요 클로이~ @anyukyung
네트워크 로직 리팩토링이 완료되어 PR 보냅니다!

🧴 PR 요약

서버 통신 로직에서 컴플리션 핸들러들이 콜백 피라미드를 만들던것을 async await를 통해 해결했습니다.
기존 레포지토리 파일들은 삭제하지 않고 deprecated를 달아주어 사용시에 warning이 발생하도록 했습니다.

📌 변경 사항

태그 DTO to Domain 매핑시에 태그의 id를 넣어주던걸 태그의 로그id를 넣어주는것으로 수정했습니다 (에러 수정)
store detail 화면에 탭바가 가려지지만 영역이 남아있는 문제를 해결했습니다.

Linked Issue

close #144

- id가 아닌 태그 id로 변환해야하는데 잘못 들어있던 것 수정

anyukyung

@Neph3779
너뭄무무너무 고생 많으셨습니다..!
말씀하셨던 대로 viewModel에 남아있는 Cancellable Task만 정리해주시면 될 것 같아요 👀👍

Neph3779 added 16 commits February 11, 2023 00:33

[Feat] #144 Async 버전 스토어 레포, 인터페이스 제작

37dd8e2

[Feat] #144 async 지원하는 이미지 업로드 메서드 제작

3560a4c

[Chore] #144 사용하지 않는 인터페이스와 유즈케이스 제작

27e237c

[Feat] #144 Async 버전 Account 레포, 인터페이스 제작

58cf317

[Feat] #144 Async 유저인포 레포지토리, 인터페이스 제작

45390e5

[Feat] #144 Async CS 레포, 인터페이스 제작

ec85c3a

[Feat] #144 Async 리뷰 업로드 레포, 인터페이스 제작

366e9ba

[Chore] #144 오타수정, 네트워크 로그 프린트문에 이모지 추가

cc49511

[Feat] #144 Account 레포 사용부분 async 버전으로 변경

b2d6f10

[Refactor] #144 Store 레포 사용처 Async 버전으로 변경

9bab1fc

[Refactor] #144 UserInfo 레포 사용처 Async 버전으로 변경

bf5a054

[Refactor] #144 CS 레포 사용처 Async 버전으로 변경

993ea88

[Refactor, Fix] #144 리뷰 레포 사용처 Async 버전으로 변경, 오타수정

a882e42

[Fix] #144 Review DTO 매핑쪽 에러 수정

ef623e0

- id가 아닌 태그 id로 변환해야하는데 잘못 들어있던 것 수정

[Fix] #144 탭바 hidden해도 영역이 남아있던 문제 해결

412f5e9

[Chore] #144 사용하지 않을 레포에 deprecate 붙이기, 폴더정리

08a275e

Neph3779 added 🔧 refactor 💻 조지 labels Feb 11, 2023

Neph3779 requested a review from anyukyung February 11, 2023 04:41

Neph3779 self-assigned this Feb 11, 2023

anyukyung requested changes Feb 11, 2023

View reviewed changes

Neph3779 added 2 commits February 11, 2023 20:46

[Chore] #144 cancel 작업 안하는 task들 삭제, Task cancel

89940e6

[Chore] #144 추천버튼 누를때 레이아웃 콘솔에러 나는 것 수정

42d8e1b

Neph3779 merged commit a623f86 into develop Feb 11, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refactor/#144 서버통신 로직 async await로 리팩토링 #146

Refactor/#144 서버통신 로직 async await로 리팩토링 #146

Neph3779 commented Feb 11, 2023 •

edited

Loading

anyukyung left a comment

Refactor/#144 서버통신 로직 async await로 리팩토링 #146

Refactor/#144 서버통신 로직 async await로 리팩토링 #146

Conversation

Neph3779 commented Feb 11, 2023 • edited Loading

🧴 PR 요약

📌 변경 사항

Linked Issue

anyukyung left a comment

Choose a reason for hiding this comment

Neph3779 commented Feb 11, 2023 •

edited

Loading