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 반영하기 #354

Merged
merged 5 commits into from
Sep 2, 2024

Conversation

JinUng41
Copy link
Contributor

@JinUng41 JinUng41 commented Sep 1, 2024

🔗 연결된 이슈

📄 작업 내용

  • Notion '1차 QA 기록' 페이지에 적인 모임 상세 화면 QA 반영
구현 내용 IPhone 13 mini
GIF
  • PrivacyInfo.plist의 테스트 토큰에 제 토큰을 넣었음에도 테스트효은으로 로그인이 되더라구요.. 꼬인건지..

💻 주요 코드 설명

QA 5가지

  • '모임 참여 인원' 우측 화살표 삭제
  • 플러스 버튼 '+' 정중앙으로 오도록 수정
  • 'D-DAY' 대문자로 표현
  • '모임 생성일' 수정된 DateFormat 반영
  • '모임 생성일 | 날짜' 간격 수정

  • 5가지에 대해서 각 커밋별로 반영하였습니다.
  • 'files changed'로 보시기 보다는 각 커밋별로 보시는 편이 좋을 것 같습니다.

👀 기타 더 이야기해볼 점

바텀 시트 애니메이션 관련

  • 애니메이션이 빈 화면(흐린 영역)을 터치할 때와 버튼(취소)를 눌렀을 때 다르게 동작하는데, 이를 개선해야 할 것 같습니다.
  • 아무래도 취소 버튼을 터치 시에는 dismiss를 호출하다 보니 그렇게 된 것 같은데, 상위 뷰컨트롤러인 BottomSheetController의 메서드를 어떻게 호출해야 할 지에 대한 고민을 해보겠습니다.
  • 아니면, 아예 새로 객체를 구현하는 방법도..

@JinUng41 JinUng41 added 🛠️ fix 버그나 오류 해결시 사용 💙 JinUng 걸스 토크에 미쳐보고 싶다면 labels Sep 1, 2024
@JinUng41 JinUng41 self-assigned this Sep 1, 2024
@JinUng41 JinUng41 linked an issue Sep 1, 2024 that may be closed by this pull request
5 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.

굿! 고생하셨습니다.

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.

짱! 고생하셨어요☺️

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.

효은아 고생했어!
LGTM

@JinUng41 JinUng41 merged commit 168a340 into suyeon Sep 2, 2024
@JinUng41 JinUng41 deleted the fix/#353-meeting-info-qa branch September 2, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ fix 버그나 오류 해결시 사용 💙 JinUng 걸스 토크에 미쳐보고 싶다면
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 모임 상세 화면 QA 반영하기
4 participants