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] 준비 정보 수정 플로우 추가 #360

Merged
merged 6 commits into from
Sep 4, 2024

Conversation

mmaybei
Copy link
Member

@mmaybei mmaybei commented Sep 3, 2024

🔗 연결된 이슈

📄 작업 내용

  • 약속 상세의 준비 현황 페이지에서 수정 버튼의 레이아웃 오류를 해결했습니다.
  • 준비 정보 수정 플로우를 추가했습니다.
  • 텍스트필드의 상태변화(filled/type/press/default), 텍스트필드 입력에 따른 완료 버튼 활성화, 준비현황에서의 수정사항 반영을 고려하여 작업했습니다.

예상한 경우의 수에 따라 작업한 세부 사항은 다음과 같습니다.

  • 수정 화면에서 이전에 입력한 값이 입력되어 있습니다.
  • 준비시간 혹은 이동시간을 0으로 입력한 경우, 즉 준비시간 혹은 이동시간이 0분인 경우(준비시간/준비분이 0인 경우 혹은 이동시간/이동분이 0) 다시 수정 화면에 진입했을 때 해당 텍스트필드는 00을 플레이스 홀더로 가진 초기 디폴트 상태로 돌아갑니다.
  • 0시간 20분, 1시간 0분과 같이 준비시간 혹은 이동시간이 0분이 아닌 경우에는 일부 텍스트필드 값이 0이더라도 위 사항(플레이스 홀더)이 반영되지 않습니다.
  • 텍스트필드를 수정하지 않아도 완료 버튼이 활성화됩니다. (아래 동작 화면에는 반영되어 있지 않습니다. 마지막 커밋을 참고해주세요!)
  • 기존 기능에서의 텍스트필드 상태는 수정 단계에서도 유효합니다.
구현 내용 iPhone 13 mini
네 텍스트 필드를 모두 수정하는 경우
일부 텍스트 필드를 수정하는 경우
네 텍스트 필드를 모두 0으로 수정하는 경우
일부 텍스트 필드를 0으로 수정하는 경우(준비시간 혹은 이동시간이 0이 되는 경우)
일부 텍스트 필드를 0으로 수정하는 경우

처음 입력하는 경우의 동작 화면입니다. 수정된 코드가 기존의 기능 구현에 영향을 미치지 않는 것을 확인했습니다.

  • 회색 텍스트 00을 플레이스 홀더로 가지고 있습니다.
  • 텍스트필드의 값이 모두 입력되었을 때 완료 버튼이 활성화됩니다.
  • 잘못된 값을 입력할 시, 토스트 메시지와 함께 텍스트필드의 값이 수정됩니다. (따라서, 특정 값을 입력했다가 지웠을 경우 완료 버튼이 활성화되지 않습니다.)
구현 내용 iPhone 13 mini
GIF

👀 기타 더 이야기해볼 점

텍스트필드의 상태변화(filled/type/press/default), 텍스트필드 입력에 따른 완료 버튼 활성화, 준비현황에서의 수정사항 반영을 중심적으로 확인해주세요! 경우의 수가 끝 없이 나오네요ㅜㅜ 혹시 제가 고려하지 못한 경우가 있다면 말씀해주시길 .. 😿

@mmaybei mmaybei added ✨ feat 기능 구현시 사용 ❤️ SuYeon 무한 잼얘 지옥에 빠지고 싶다면 labels Sep 3, 2024
@mmaybei mmaybei self-assigned this Sep 3, 2024
Copy link
Member

잘햇어요 북북

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.

잘했어요 벅벅

@mmaybei mmaybei linked an issue Sep 3, 2024 that may be closed by this pull request
2 tasks
Copy link
Member

@youz2me youz2me left a comment

Choose a reason for hiding this comment

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

복복 수야미 고마워요 !! 🥹🤍

@@ -68,6 +66,7 @@ class ReadyPlanInfoView: BaseView {
editButton.snp.makeConstraints {
$0.trailing.equalToSuperview().inset(22)
$0.bottom.equalToSuperview().inset(18)
$0.width.equalTo(Screen.width(60))
Copy link
Member

Choose a reason for hiding this comment

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

어쩐지 안보이더라니 ... 감사합니다 ㅎㅎ

@mmaybei mmaybei merged commit b2e8a13 into suyeon Sep 4, 2024
@mmaybei mmaybei deleted the feat/#358-editReadyInfo branch September 5, 2024 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 구현시 사용 ❤️ SuYeon 무한 잼얘 지옥에 빠지고 싶다면
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 준비 정보 입력 수정 플로우 추가
4 participants