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

#43 - 계정 관리 화면 구현 #45

Merged

Conversation

honghoker
Copy link
Member

🔗 관련 이슈

🛠️ 작업한 내용

  • 마이페이지 탭의 계정 관리 화면을 구현했습니다.

📸 스크린샷

데모
Simulator Screen Recording - iPhone 15 Pro - 2025-01-21 at 02 36 59

🚨 참고 사항

흑흑... 정말 단순한 작업인 줄 알았는데 생각보다 오래 걸렸습니다.
로그아웃이랑 회원 탈퇴 후 rootView로 이동하는 로직을 구성했는데, 이동할 때마다 보라색 경고가 뜨더라고요.
알고 보니, rootView로 전환되는 과정에서 MyPageViewonAppear가 호출되면서 불필요한 액션을 send하고 있었습니다.
이미 사라진 Store에 액션을 보내니 당연히 올바른 동작이 아니라고 경고가 표시되는 거였어요!

예외 처리를 해서 경고는 사라졌는데, 이런 방식이 마음에 들진 않네요.
UIKit 쓸 때가 그립기도 하고 사실 그냥 TCA가 미운걸지도 모르겠습니다.

조금 더 좋은 방법이 있는지 찾아봐야 될 것 같네용
개선할 부분있으면 코멘트 부탁드림니다!

@honghoker honghoker added ✨ 기능추가 새로운 기능 추가 🎨 디자인 UI 디자인 작업 labels Jan 20, 2025
@honghoker honghoker requested a review from sunny5875 January 20, 2025 17:53
@honghoker honghoker self-assigned this Jan 20, 2025
Copy link
Collaborator

@sunny5875 sunny5875 left a comment

Choose a reason for hiding this comment

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

드디어 애증의 보라색 경고를 해결하셨군요... 짱드립니다🚀🚀🚀

public struct State {
@Shared(.userInfo) var userInfo: UserInfo?
@Presents var sheet: ConfirmationSheetStore.State?
var presentedSheet: SheetType?
Copy link
Collaborator

Choose a reason for hiding this comment

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

@presents를 쓰는 방법도 있었군요! 덕분에 알게되네요:) 저도 한번 써봐야겠습니당

Copy link
Member Author

Choose a reason for hiding this comment

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

맞아요!
이번에 써보면서 느낀건 View에서 사용할 Store를 별도로 만들어줘야해서, DesignSystem 모듈에 넣자니 TCA 의존성이 생기게되어서 애매하더라구요.
만약 현재 구조에서 해당 sheet를 다른 View에서도 쓸거면 DesignSystem 모듈에 넣고 걷어내야될 것 같습니다.. ㅋㅋㅋ

@honghoker honghoker merged commit 2462c4d into DDD-Community:develop Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ 기능추가 새로운 기능 추가 🎨 디자인 UI 디자인 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 마이페이지 - 계정 관리 화면 구현
2 participants