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.

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

[Feat] #32 Home API 연결 및 바인딩 #33

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

cchanmi
Copy link
Collaborator

@cchanmi cchanmi commented Sep 21, 2024

작업한 내용

  • Home API 연결하고 바인딩까지 진행했습니다.
  • 더미로 가지고 있던 지도 데이터, 전역으로 여러 곳에서 영향을 주고 있던 데이터들 전부 제거했습니다. 데이터 흐름이 깔끔해졌어요.
  • BaseRequest 구조를 변경하고, 이제 Result 타입으로 값을 받지 않아서 ReactorKit에서 Error 타입을 MDError로 따로 캐스팅 해 주어야 하기 때문에 handleError 메서드도 추가했습니다. 5f00ff7
  • 코드 확인해 주시고 이해 안 되는 부분 리뷰 남겨 주세요.

스크린샷

설명
스크린샷 2024-09-21 오후 10 22 43 경기도, 서울, 강원도, 제주도에 방문 기록이 있는 모습!

관련 이슈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Home API 연결
1 participant