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

[Fix] #114 - 서버통신 로직 수정 및 UserManager 클래스 구현 #117

Merged
merged 2 commits into from
Jul 18, 2022
Merged

[Fix] #114 - 서버통신 로직 수정 및 UserManager 클래스 구현 #117

merged 2 commits into from
Jul 18, 2022

Conversation

L-j-h-c
Copy link
Member

@L-j-h-c L-j-h-c commented Jul 18, 2022

🔥 Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

  • 서버통신 관련 URL이 연결되지 않는 이슈가 발견되어서 식당 상세정보 요약 불러오기 API를 연결하면서 원인을 파악해 보았습니다.
    BaseURL이 잘못되어 발생한 이슈였습니다^__^
  • 추가로 자동로그인 구현을 위해 UserDefaultWrapper와 UserManager 클래스를 구현했습니다.
  • UserDefaultWrapper는 UserDefault의 사용을 간편하게 해주는 Property Wrapper의 역할을 담당하고, 이러한 Wrapper를 UserManager 내에 사용하여 앱 전역에서 유저 정보, 토큰 관리를 할 수 있도록 합니다.

🚨 참고 사항

  • 참고 : Property Wrapper란
  • 유저디폴트가 자동로그인에서 필요한 이유 : 사용자의 기기에 저장되는 유저디폴트를 이용하여 로그인 경험 및 소셜로그인 종류 등의 정보를 저장해 놓을 수 있다. 유저디폴트 값에 따라 자동로그인을 해도 되는지, 온보딩을 보여줘야 하는지, 새로운 로그인 request가 필요한지에 대해 판단할 수 있다.
    @yungu0010 자동로그인 해줭

📟 관련 이슈

@L-j-h-c L-j-h-c added Feat 새로운 기능 구현 Fix 오류 해결, 코드 수정 🧑🏻‍💻Duno labels Jul 18, 2022
@L-j-h-c L-j-h-c self-assigned this Jul 18, 2022
@L-j-h-c L-j-h-c changed the title [Fix] #114 - 서버통시 로직 수정 및 UserManager 클래스 구현 [Fix] #114 - 서버통신 로직 수정 및 UserManager 클래스 구현 Jul 18, 2022
0lynny
0lynny approved these changes Jul 18, 2022
View reviewed changes
Copy link
Member

@0lynny 0lynny left a comment

Choose a reason for hiding this comment

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

서버통신 화이팅.....! 수고하셨슴다~^^

yungu0010
yungu0010 approved these changes Jul 18, 2022
View reviewed changes
Copy link
Member

@yungu0010 yungu0010 left a comment

Choose a reason for hiding this comment

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

아 ㅋ ㅋ 자동로그인 딱 대

@L-j-h-c L-j-h-c merged commit 4df054f into Health-Food-Me:develop Jul 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0lynny 0lynny 0lynny approved these changes

@yungu0010 yungu0010 yungu0010 approved these changes

@i-colours-u i-colours-u Awaiting requested review from i-colours-u

@0inn 0inn Awaiting requested review from 0inn

Assignees

@L-j-h-c L-j-h-c

Labels
🧑🏻‍💻Duno Feat 새로운 기능 구현 Fix 오류 해결, 코드 수정
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Fix] 서버통신 로직 수정
3 participants
@L-j-h-c @yungu0010 @0lynny