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] #339 콕 찌르기 피쳐 진입 플로우 #340

Conversation

lsj8706
Copy link
Member

@lsj8706 lsj8706 commented Dec 27, 2023

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

  • 앱의 메인 화면(홈 화면)에서 콕 찌르기 피쳐로 진입하는 플로우를 구현했습니다.
  • 우선 비활동 회원의 경우 아직 콕 찌르기 피쳐 사용 대상자가 아니기 때문에 메인 화면에서 콕 찌르기로 이동할 수 있는 버튼을 숨겼습니다.
  • 활동 회원은 콕 찌르기 피쳐의 이용이 가능합니다.
  • 메인 화면에서 콕 찌르기 피쳐로 이동하는 버튼(Cell) 클릭 시 서버와 통신하여 해당 유저가 콕 찌르기 온보딩 대상 유저인지 확인합니다.
  • 맞다면 온보딩 뷰로 아니라면 콕 찌르기 메인 뷰로 이동합니다.
  • 이 과정에서 서버 통신 시간이 길어질 수 있기 때문에 Loading Indicator가 나오도록 했습니다.

📌 참고 사항

  • MainViewModel에서 cell 터치 이벤트를 받는 로직의 코드가 깔끔하지 않은 것 같아서 이 부분 코드 다듬는 작업도 수행했습니다.

📸 스크린샷

기능 스크린샷
�콕 찌르기로 이동(온보딩) Simulator Screen Recording - iPhone 14 - 2023-12-27 at 18 23 59
�콕 찌르기로 이동(메인) Simulator Screen Recording - iPhone 14 - 2023-12-27 at 18 25 00

📮 관련 이슈

@lsj8706 lsj8706 added Feat 새로운 기능 구현 세진🤝🏻 labels Dec 27, 2023
@lsj8706 lsj8706 self-assigned this Dec 27, 2023
Copy link
Contributor

@elesahich elesahich left a comment

Choose a reason for hiding this comment

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

고생했습니다~~ 감사합니다

// UserDefaultKeyList.User.isFirstVisitToPokeView = false
// self?.onFirstVisitInOnboarding?()
self?.onFirstVisitInOnboarding?()
UserDefaultKeyList.User.isFirstVisitToPokeView = false
Copy link
Contributor

Choose a reason for hiding this comment

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

여기랑 L:57은 사용 안하게 되니 요것도 없애줄수 있엉? UserDefaultKeyList~ 여기 선언부도 오네가이시맛스!

Copy link
Member Author

Choose a reason for hiding this comment

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

그럼 여기 1번 영역 바텀 시트가 매번 나올텐데 우리 기획에 어떤 게 맞는건지 잘 모르겠어서 일단 안 지웠어!
온보딩 뷰가 뜨면 무조건 바텀시트도 떠야하는건가? 아니면 사용자에게 1회만 바텀시트가 나오는걸까?

Copy link
Contributor

Choose a reason for hiding this comment

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

(구컴완)

@lsj8706 lsj8706 merged commit bec34e8 into sopt-makers:develop Dec 27, 2023
@lsj8706 lsj8706 deleted the feat/#339-콕-찌르기-피쳐-진입-플로우 branch December 27, 2023 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 구현 size/L 세진🤝🏻
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 콕 찌르기 피쳐 진입 플로우
2 participants