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

Feat [#58] SignUpManager, 이미지 추가 #72

Merged
merged 12 commits into from
Jan 13, 2024
Merged

Conversation

kim-seonwoo
Copy link
Member

👾 작업 내용

SignUpManager 파일을 추가하였습니다.

  • 회원 가입 시 활용되는 정보들을 각 온보딩 단계에서 저장해야 했습니다.
  • 싱글톤 객체를 형성하여 이에 저장하고 마지막에 API에 쏴주는 방식이 좋다고 생각했습니다.
  • 다음과 같이 접근하여 저장합니다.
        SignUpManager.shared.goalTime = totalTime

스플래시 뷰, 로그인 뷰, 회원가입 완료 화면에 이미지를 추가하였습니다.

  • splash의 경우 레이아웃을 구현하였습니다.

온보딩 과정 단계 중 String을 받아오는 로직을 추가하였습니다. (프로토콜 이용)

  • 타임 피커와 HMHSelectButton에서 delegate 으로 넘겨줍니다.
extension SelectTotalTimeController: TimePickerDelegate {
    func updateAvailability(selectedValue: Int) {
        nextButton.updateStatus(isEnabled: true)
        totalTime = selectedValue
    }
}

🚀 PR Point

  • delegate가 최선의 방법일까요?

📸 스크린샷

구현 내용 스크린샷
스플래시 image

🚀 기기 대응

기기명 Iphone 13 mini Iphone 14 Iphone 15 pro Iphone SE(3rd)
스크린샷 image image image image

✅ Issue

Resolved #58

@kim-seonwoo kim-seonwoo added 😎선우 선우의 issue 🌈 feat 기능 구현 labels Jan 13, 2024
@kim-seonwoo kim-seonwoo added this to the 🚀1차 스프린트🚀 milestone Jan 13, 2024
@kim-seonwoo kim-seonwoo self-assigned this Jan 13, 2024
@kim-seonwoo kim-seonwoo linked an issue Jan 13, 2024 that may be closed by this pull request
Copy link
Member

@Zoe0929 Zoe0929 left a comment

Choose a reason for hiding this comment

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

P5 아주 Gooooood 입니다.

Copy link
Contributor

@boyeon0119 boyeon0119 left a comment

Choose a reason for hiding this comment

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

헤이 써늬~ 네이스~

Comment on lines 80 to 81
let nextViewController = SelectTotalTimeController()
self.navigationController?.pushViewController(nextViewController, animated: false)
Copy link
Contributor

Choose a reason for hiding this comment

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

p5. 이거 뷰컨이면 당신이 만든 익스텐션 가능..? 좀 다른건가

Copy link
Member Author

Choose a reason for hiding this comment

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

제가 만든 익스텐션은 rootViewController 자체를 변경함으로써 스택으로 쌓이는 push방법과는 다릅니다.
해당 경우에는 이전 뷰 컨트롤러로 돌아오는 기능이 필요합니다.

따라서 push 방법을 그대로 사용하였습니다.

충분한 답변이 되셨나요?

@kim-seonwoo kim-seonwoo merged commit a527e2c into develop Jan 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌈 feat 기능 구현 😎선우 선우의 issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Feat] 온보딩 설문 단계에서 저장 로직 구현
3 participants