[FEAT] 프로필 설정 UI 구현 #103
[FEAT] 프로필 설정 UI 구현 #103
Conversation
| kusitmsComposableWithAnimation(NavRoutes.ProfileEdit.route) { | ||
| ProfileEditScreen( | ||
| onBack = { navController.navigateUp() } | ||
| ) | ||
| } |
There was a problem hiding this comment.
제가 이전 코드를 짠 것을 사용하는데에 문제가 있었는지 혹시 궁금합니다 🥹 .... 변경 사항 보고 놀래서 들어왔는데 재사용을 안하시고 새로 만드셨길래
There was a problem hiding this comment.
민서님 코드 재사용 하려고 했는데, Composable 선언할 때 파라미터로 SignInViewModel을 받으셨더라구요!
그래서 저도 SignInViewModel을 써야하거나, 뷰를 새로 만들던가 해야했는데
민서님 코드와 달리 프로필 설정 부분에서 이메일/전화번호 수정이 가능하기도 했고
Profile 설정쪽 뷰모델을 아예 다르게 지정하는게 맞을 것 같아서 뷰를 새로 만들고 ProfileEditViewModel을 파라미터로 설정했습니당
ProfileEditViewModel에서 사용하는 API, SignInViewModel에서 사용하는 API가 달라 분리를 하긴 했습니다.!
이 부분에 대해서는 저도 얘기를 드리고 싶었는데, 차라리 ViewModel을 하나로 통일하고 재사용하는 식으로 리팩토링 하는게 나을 것 같아요! 피그마상에서도 자잘하게 다른 부분이 꽤나 있어 추후 작업이 필요해 보입니당.. 그래도 ViewModel이 없는 쪽에서는 최대한 민서님 코드를 재사용 해둔 것 같아요 이 부분에 대해서 저희끼리 회의할 때 얘기하면 좋을 것 같습니다!
There was a problem hiding this comment.
같은 뷰모델을 사용해도 될것이라 생각했는데 이전 화면에는 필요없는 api도 같이 딸려 들어있다보니 조금 낭비적인 측면도 있긴 하겠네요 .. 🤔
추후에 아린님 말씀처럼 공통 뷰 모델로 통일하는 방안이 좋은 해결 방안일것 같습니다 출시 후에 논의해봐요
답변 감사합니다!
Issue number and Link
이슈 번호 : #101
Summary
내 프로필 설정 UI를 구현하였습니다
민서님이 짜두신 초기 회원 프로필 설정 코드를 많이 참고하였습니다!
PR Type
Other Information
Common Type
branch