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

마이페이지-설정 디자인 & 기능 일부 #92

Merged
merged 20 commits into from
Sep 6, 2023

Conversation

enebin
Copy link
Member

@enebin enebin commented Sep 4, 2023

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-09-04.at.19.58.08.mp4
  • 로그아웃만 구현 된 상태
  • 푸시는 붙이긴 했는데 확인 못함
  • 탈퇴는 API 나올 때까지 대기

기타

  • 웹 뷰 로딩 속도 개선

Copy link
Member Author

@enebin enebin left a comment

Choose a reason for hiding this comment

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

Big PR 죄송합니다,,

Copy link
Member Author

@enebin enebin Sep 4, 2023

Choose a reason for hiding this comment

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

가끔 스택 넘칠 때가 있는데 그 때 이걸로 감싸주면 '임시로' 해결 되는 듯함. 첨부터 스토어 한 개만 있게끔 설계하는게 베스트긴 한 것 같은데 내가 지금 숙련도가 딸려서;; 스택 터지는데 고칠 시간 없을 때 최후의 보루로서 사용해주심 될듯

Copy link
Member Author

Choose a reason for hiding this comment

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

웹 뷰 헤더에 토큰을 넣어달라는 요청이 있었음. 이건 디코 프론트엔드 요청 쓰레드 보면 나와있음.
웹 쪽에선 지금 헤더를 못 받고 있다고 해서 아직 이게 스펙으로 확정되진 않을 수도 있음.

Copy link
Member Author

Choose a reason for hiding this comment

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

푸시토큰 등록/해제 관리하는 클래스임.

Copy link
Member Author

Choose a reason for hiding this comment

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

SwitchStore를 써서 로직을 좀 간단하게 만들었음.

@enebin enebin requested review from 0inn and k906506 September 4, 2023 11:06
@enebin enebin self-assigned this Sep 4, 2023
@enebin enebin marked this pull request as ready for review September 5, 2023 01:53
Copy link
Contributor

@0inn 0inn left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ㅏ. . 저도 후딱 달려보겠읍니다.

/// 스택 오버플로우 방지용 래퍼
/// 참고: https://github.com/pointfreeco/swift-composable-architecture/discussions/488
@propertyWrapper
public struct Box<T: Equatable>: Equatable {
Copy link
Contributor

Choose a reason for hiding this comment

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

오우 . . 첨보는 아이 굳입니다.

@enebin enebin merged commit 6662b61 into refactor/onboarding-design(#87) Sep 6, 2023
1 check passed
@enebin enebin deleted the feature/setting-view(#89) branch September 6, 2023 01:50
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.

2 participants