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

[fix] 최종 QA 반영 #393

Merged
merged 3 commits into from
Sep 22, 2024
Merged

[fix] 최종 QA 반영 #393

merged 3 commits into from
Sep 22, 2024

Conversation

youz2me
Copy link
Member

@youz2me youz2me commented Sep 20, 2024

🔗 연결된 이슈

📄 작업 내용

  • 준비 현황 플로우 텍스트를 수정했습니다.
  • 꾸물거릴 시간이 없어요! 팝업을 도착 완료 시에만 숨겨지도록 구현했습니다.
  • 약속 정보 뷰 시간 표현 형식을 12시간 형식으로 교체했습니다.
구현 내용 IPhone 15 pro
GIF

💻 주요 코드 설명

꾸물거릴 시간이 없어요! 팝업

  • 현재 ViewModel에서 서버로부터 받아오는 preperationStartAt, departureAt, arrivalAt 값 유무에 따라 나의 준비 상태를 none, ready, move, done으로 바인딩하도록 처리해두었습니다.
  • 그래서 팝업 숨김 처리는 그냥 state 케이스에 나누어 처리하는 방식으로 구현했습니다. 이후 실시간으로 지각 여부를 판단할 때도 이 방식을 조금 더 디벨롭하는 식으로 가져갈 것 같습니다.
ReadyStatusViewController.swift
             owner.rootView.popUpImageView.do {
                switch state {
                case .none, .ready, .move:
                    $0.isHidden = false
                case .done:
                    $0.isHidden = true
                }
            }

@youz2me youz2me added ✨ feat 기능 구현시 사용 🛠️ fix 버그나 오류 해결시 사용 💚 YouJin 저댄고?저댄고?저댄고?저댄고?저댄고? labels Sep 20, 2024
@youz2me youz2me self-assigned this Sep 20, 2024
@youz2me youz2me linked an issue Sep 20, 2024 that may be closed by this pull request
1 task
Copy link
Contributor

@JinUng41 JinUng41 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~

Copy link
Member

@hooni0918 hooni0918 left a comment

Choose a reason for hiding this comment

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

잘했어요~

Copy link
Member

@mmaybei mmaybei left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@youz2me youz2me merged commit 7486c03 into suyeon Sep 22, 2024
@youz2me youz2me deleted the fix/#392-promise-qa branch September 22, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 구현시 사용 🛠️ fix 버그나 오류 해결시 사용 💚 YouJin 저댄고?저댄고?저댄고?저댄고?저댄고?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 최종 QA 반영
4 participants