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

DMVM-208(feat): network manager 구현 #23

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

Conversation

bokoo14
Copy link
Collaborator

@bokoo14 bokoo14 commented Jan 25, 2025

🎫 지라 티켓 이슈

https://dogbusiness.atlassian.net/browse/DMVM-208

🛠️ 작업 내용

DMVM-208(feat): 네트워크 매니저 개발

[기존]

  • API 호출 시 각 메서드마다 네트워크 코드를 개별 작성을 하는 방식으로 되어 있어, 네트워킹이 필요할 경우 네트워크 관련 코드를 항상 작성해야 하므로 중복이 발생하고, 코드의 재사용성과 유지보수성이 낮음

[신규 기능]

  • 네트워크 매니저(NetworkImp)를 설계하여 네트워크 호출 코드를 표준화하여, request, response의 DTO만 관리하도록 구현하여, NetworkImp의 request 메서드만 호출하면 되도록 구현하였음
  • URLSession을 기반으로 제네릭(Request/Response) 구조를 적용해 다양한 API를 유연하게 처리되도록 설계

DMVM-208(fix): 카카오톡 로그인 오류 수정

기존 정상 동작하던 카카오톡 로그인 기능에서 BE의 response 규격화를 적용함으로써, DTO가 맞지 않아 decoding이 정상적으로 되지 않는 버그를 수정했습니다.

참고 PR: PBTP/mongle-server#51
[신규 response DTO]

export class ResponseEntity<T> {
  data?: T;
  message: string;
  statusCode: number;
  secretMessage?: string;
}

DMVM-228(chore): 로그인 화면 수정

기존의 Lo-Fi에서 작업되었던 화면을 최신의 Hi-Fi에 맞게 수정하였습니다.



📱 작업 화면

화면 이름 스크린샷
ViewName

🗒️ Note (optional)

response가 변경됨에 따라 response가 decoding이 되지 않아 카카오톡 로그인 후 화면 이동이 되지 않는 버그를 수정했습니다.
@bokoo14 bokoo14 requested a review from bulmang January 25, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants