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

[JC/Week9] Version 1.1 - AuthViewModel 최적화 #67

Merged
merged 19 commits into from
Jun 19, 2024
Merged

Conversation

jayn2u
Copy link
Member

@jayn2u jayn2u commented Jun 9, 2024

📍Summary

해당 PR에 대한 내용을 요약해주세요.

💡PR Point

코드들에서 핵심 코드들을 설명해주세요.
또는 알게된 부분들을 공유해주세요.

  • AuthViewModel을 최적화하면서 AuthView에서 불필요한 의존성을 낮추고 유지보수성을 높이려고 해보았습니다.

🤔 Question

작업하시면서 궁금했던 질문들을 남겨주세요.

  • 백엔드 개발자 없이 클라이언트에서만 사용자 인증을 구현하려고 하다보니 BaaS(Backend as a Service) 제품(like Firebase, Supabase)에서 제공하는 SDK를 사용하니 백엔드 개발자가 있는 협업에서는 어떻게 해야 할 지 아직 모르겠네요.
  • BaaS SDK는 클라이언트 개발자가 불필요하게 토큰을 핸들링할 필요도 없더라구요.
  • 이 시점에서 조언이 궁금합니다!

🔑 Simulator

simulator 영상을 화면 녹화 후 보여주세요.

@jayn2u jayn2u added the ch9 label Jun 9, 2024
@jayn2u jayn2u added this to the iOS 스터디 milestone Jun 9, 2024
@jayn2u jayn2u self-assigned this Jun 9, 2024
@jayn2u
Copy link
Member Author

jayn2u commented Jun 9, 2024

🤔 Discussion

AuthViewModel을 사용하는데, 있어서 body 프로퍼티에서 객체의 참조 문제가 있습니다. 사용자 정보를 담고 있는 객체는 앱 전반에서 동작해야 하는 코드인데, AuthView에서만 사용하면 다른 View에서 객체의 참조 문제가 있을 것 같다는 생각이 듭니다.

그래서 이 시점에서 EnviromentObject 프로퍼티 래퍼를 사용해서 환경 변수로서 동작해야 하지 않나라는 생각이 듭니다.
이 부분에서는 추후 업데이트하겠습니다.

현재 문제의 원인을 어느정도 파악했기에 환경변수에 대해서 공부를 하고 업데이트된 코드를 반영하겠습니다.

@jayn2u jayn2u merged commit f419c69 into main Jun 19, 2024
@jayn2u jayn2u deleted the JC/Week9-Version-1.1 branch June 19, 2024 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Week9] Token
1 participant