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.

Sign up for GitHub

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

Jump to bottom

[refactor] 홈 화면 기능 QA 및 수정된 서버 명세 반영 #335

Merged
merged 16 commits into from
Aug 26, 2024
Merged

[refactor] 홈 화면 기능 QA 및 수정된 서버 명세 반영 #335

merged 16 commits into from
Aug 26, 2024

Conversation

mmaybei
Copy link
Member

@mmaybei mmaybei commented Aug 26, 2024

🔗 연결된 이슈

📄 작업 내용

  • 기존에 아래로 스크롤 시 바운싱되지 않던 문제를 해결하였습니다.
  • 오늘의 약속에서 준비 상태 버튼을 눌렀을 때, 페이지를 이동해야 반영되던 time 레이블이 바로 반영될 수 있도록 수정하였습니다.
  • 오늘의 약속에서 버튼 레이블이 반영되지 않던 오류를 수정하였습니다.
  • 오늘의 약속에서 새로운 오늘의 약속이 추가되었을 때 (예를 들면 오늘 오후 4시의 약속이 있는 상태에서 오늘 오후 2시의 약속이 추가되어 오늘의 약속 뷰가 변경될 때) 준비 상태 버튼이 새롭게 리셋되지 않던 오류를 수정하였습니다.
  • 수정된 서버 명세에 따라 DTO를 수정하고 데이터를 가공하여 새롭게 바인딩하였습니다.
  • 서비스타입 변경사항을 반영하였습니다.
  • 외에도 간단한 코드 수정을 진행하였습니다.
구현 내용 IPhone 15 pro
GIF

👀 기타 더 이야기해볼 점

현재 레이블 하이라이트 익스텐션에서 "1번의 약속에서 1번 지각했어요"와 같이 두 "1번"을 모두 하이라이트해야 하는 경우를 반영하지 못하는 것 같습니다. 해당 오류는 이후 PR에서 반영하도록 하겠습니다.

@mmaybei
fix/#301 스크롤뷰 바운스 관련 이슈 해결
a4bb97b
@mmaybei
fix/#301 수정된 API 명세 반영
7e95f19
@mmaybei
refactor/#301 서비스타입 변경사항 반영
659f7b3
@mmaybei
refactor/#301 서비스프로토콜 관련 수정
b0468e6
@mmaybei
refactor/#301 아이콘 크기 및 색상 변경
44ed84a
@mmaybei
refactor/#301 현재 상태 관련 뷰모델 로직 수정
a22fdee
@mmaybei
chore/#301 불필요한 주석 삭제 및 마크주석 추가
e73dae2
@mmaybei
fix/#301 오늘의 약속 디자인 반영
e60d1a0
@mmaybei
chore/#301 바인딩 함수 네이밍 변경
82757d8
@mmaybei
chore/#301 간단한 코드 수정
b49b79e
@mmaybei
chore/#301 간단한 코드 수정
0ec8ec2
@mmaybei
fix/#301 준비 상태 관련 오류 수정
f750f3e 
새로운 약속이 추가되어 오늘의 약속이 바뀔 때 준비 상태가 바뀌지 않는 오류 및 페이지 이동 시에만 시간 레이블이 반영되는 오류 수정
@mmaybei
refactor/#301 수정된 서버 명세에 따른 데이터 포맷 및 바인딩
e53b252
@mmaybei
chore/#301 불필요한 주석 삭제
d83b3bf
@mmaybei
refact/#301 셀 디데이 레이블 수정
15c78cb
@mmaybei mmaybei added ♻️ refactor 기존 코드를 리팩토링하거나 수정하는 등 사용 (생산적인 경우) ❤️ SuYeon 무한 잼얘 지옥에 빠지고 싶다면 labels Aug 26, 2024
@mmaybei mmaybei self-assigned this Aug 26, 2024
@mmaybei mmaybei linked an issue Aug 26, 2024 that may be closed by this pull request
[refactor] 홈 뷰 전반적인 리팩토링 #301
Closed
4 tasks
JinUng41
JinUng41 approved these changes Aug 26, 2024
View reviewed changes
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.

고생하셨습니다~

youz2me
youz2me approved these changes Aug 26, 2024
View reviewed changes
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.

LGTM 👍 수야미 화이팅!

hooni0918
hooni0918 approved these changes Aug 26, 2024
View reviewed changes
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.

와멋져요엘지티엠벅벅

@youz2me
Merge remote-tracking branch 'refs/remotes/origin/suyeon'
0248303 
Conflicts:
	KkuMulKum.xcodeproj/project.pbxproj
@mmaybei mmaybei merged commit d04adba into suyeon Aug 26, 2024
@JinUng41 JinUng41 deleted the refactor/#301-home branch September 13, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@youz2me youz2me youz2me approved these changes

@JinUng41 JinUng41 JinUng41 approved these changes

@hooni0918 hooni0918 hooni0918 approved these changes

Assignees

@mmaybei mmaybei

Labels
♻️ refactor 기존 코드를 리팩토링하거나 수정하는 등 사용 (생산적인 경우) ❤️ SuYeon 무한 잼얘 지옥에 빠지고 싶다면
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[refactor] 홈 뷰 전반적인 리팩토링
4 participants
@mmaybei @youz2me @JinUng41 @hooni0918