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

[REFACTOR] api 로직 수정 #345

Merged
merged 28 commits into from
Apr 30, 2024
Merged

[REFACTOR] api 로직 수정 #345

merged 28 commits into from
Apr 30, 2024

Conversation

dongx0915
Copy link
Member

📌 개요

✨ 작업 내용

  • Api 요청 로직 Flow로 변환
  • ResponseInterceptor BaseResponse 체크 로직 수정

✨ PR 포인트

  • 모든 코드를 보긴 힘들 것 같으니 앱 동작에서 문제 없는지를 봐주시면 좋을 것 같습니다.
  • ResponseInterceptor에서 BaseResponse 체크 로직 부분은 코드 확인 한 번만 해주세요~
    • BaseResponse로 json 파싱 했을 때 BaseResponse 형태가 아니어도 예외가 발생하지 않아서 속성이 존재하는지 체크하는 방법으로 수정했습니다.

📸 스크린샷/동영상

dongx0915 added 27 commits March 28, 2024 01:40
@dongx0915
[refactor] 추천 코스 요청 코드 Flow로 변경
cd2bc5c
@dongx0915
[refactor] 내 업로드 코스 요청 코드 flow로 변환
525afae
@dongx0915
[fix] 추천 코스 새로고침 시 크래시 수정
c632952 
* Flow로 변환 후 크래시 발생하여 임시로 코드 추가
@dongx0915
[refactor] 코스 스크랩 부분 Flow로 변환
105ae77
@dongx0915
[refactor] 코스 업로드 요청 부분 Flow로 변환
487ee2b
@dongx0915
[refactor] 코스 검색 요청 부분 Flow로 변환
ead78fb
@dongx0915
[refactor] 유저 정보 조회 요청 부분 Flow로 변환
77cc612
@dongx0915
[refactor] 업로드한 코스 요청 부분 Flow로 변환
32eedc8
@dongx0915
[refactor] 코스 삭제 요청 부분 Flow로 변환
e6b3b84
@dongx0915
[refactor] 회원 탈퇴 요청 부분 Flow로 변환
c9ceed7
@dongx0915
[refactor] 스탬프 조회 요청 부분 Flow로 변환
bfcf0a9
@dongx0915
[refactor] 러닝 기록 조회 요청 부분 Flow로 변환
4fa5433
@dongx0915
Merge branch 'develop' into feature/refactor-api-flow-discover-mypage
1f670be
@dongx0915
[refactor] #344 기록 코스 삭제 부분 Flow로 변환
b7e95ab
@dongx0915
[refactor] #344 기록 코스 제목 수정 부분 Flow로 변환
7336e1a
@dongx0915
[refactor] #344 닉네임 수정 부분 Flow로 변환
cde622f
@dongx0915
[refactor] #344 로그인 요청 부분 Flow로 변환
d367229
@dongx0915
[refactor] #344 코스 상세 조회 부분 Flow로 변환
af3c8da 
* 미로그인 상태로 딥링크 접속시 TASk 처리 추가
@dongx0915
[refactor] #344 코스 수정 부분 Flow로 변환
f8a5d67
@dongx0915
[refactor] #344 내가 그린 코스 삭제 부분 Flow로 변환
bdb2560
@dongx0915
[refactor] #344 collect -> collectResult 수정
f33d8a8
@dongx0915
Merge branch 'feature/refactor-api-flow-draw-course' of into feature/…
77d600b 
…refactor-api-flow-discover-mypage
@dongx0915
[refactor] #344 유저 프로필 조회 Flow로 변환
6e0e57d
@dongx0915
[fix] #344 ResponseInterceptor BaseResponse 체크 구문 수정
012e3e2
@dongx0915
[refactor] #344 CourseService V2 명칭 변경
5c31c0e 
* 리팩토링 완료로 인해 V2 -> V1으로 변경
@dongx0915
[refactor] #344 LoginService V2 명칭 변경
3a7a767 
* 리팩토링 완료로 인해 V2 -> V1으로 변경
@dongx0915
[refactor] #344 코드 정리
09c0633
@dongx0915 dongx0915 added 우남 🐼 우남 담당 REFACTOR 🧹 코드 리팩토링 동현 🐨 동현 담당 labels Apr 13, 2024
@dongx0915 dongx0915 requested review from leeeha, sxunea and unam98 April 13, 2024 07:38
unam98
unam98 approved these changes Apr 19, 2024
View reviewed changes
Copy link
Contributor

@unam98 unam98 left a comment

Choose a reason for hiding this comment

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

수고하셨습니당!!

leeeha
leeeha approved these changes Apr 23, 2024
View reviewed changes
Copy link
Member

@leeeha leeeha left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

sxunea
sxunea approved these changes Apr 24, 2024
View reviewed changes
Copy link
Collaborator

@sxunea sxunea left a comment

Choose a reason for hiding this comment

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

고생했습니다 ~!

@dongx0915 dongx0915 merged commit 2f2fa5d into develop Apr 30, 2024
2 checks passed
dongx0915 added a commit that referenced this pull request May 2, 2024
@dongx0915
Revert "Merge pull request #345 from Runnect/feature/refactor-api-flo…
d8dd161 
…w-discover-mypage"

This reverts commit 2f2fa5d, reversing
changes made to 5661733.
dongx0915 added a commit that referenced this pull request May 2, 2024
@dongx0915
Merge pull request #345 from Runnect/feature/refactor-api-flow-discov…
2176bd5 
…er-mypage

[REFACTOR] api 로직 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unam98 unam98 unam98 approved these changes

@sxunea sxunea sxunea approved these changes

@leeeha leeeha leeeha approved these changes

Assignees

@dongx0915 dongx0915

@unam98 unam98

Labels
REFACTOR 🧹 코드 리팩토링 동현 🐨 동현 담당 우남 🐼 우남 담당
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[REFACTOR] api 로직 수정
4 participants
@dongx0915 @leeeha @sxunea @unam98