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 반영 #374

Merged
merged 9 commits into from
Sep 10, 2024
Merged

[refactor] 모임 내 약속 조회 기능 수정 및 QA 반영 #374

merged 9 commits into from
Sep 10, 2024

Conversation

JinUng41
Copy link
Contributor

@JinUng41 JinUng41 commented Sep 9, 2024 •
edited
Loading

🔗 연결된 이슈

📄 작업 내용

  • 모임 상세 화면 내, 약속을 조회하는 방식을 변경하였습니다. (세그먼트가 변경될 때마다 새로 네트워크 요청)
  • 약속 상세 화면에서 다시 돌아왔을 때, 프로필 이미지가 확대되어 있는 현상을 제거하였습니다.
구현 내용 IPhone 11 pro max
수정 전, 모임 상세
수정 전, 이용약관/문의하기
수정 후, 모임 상세
수정 후, 이용약관/문의하기

👀 기타 더 이야기해볼 점

  • 변경에 상관없이 세그먼트가 터치될 때에도 대응을 해보고 싶었는데 실패했습니다.

@JinUng41
fix/#373 프로비저닝 파일 수정
7e72d63
@JinUng41
refactor/#373 네비게이션 바 배경 흰색 지정
845a559
@JinUng41
chore/#373 'self'키워드 삭제
11d4355 
- 옵셔널 바인딩을 한 'self'는 코드 블럭 영역 내에서 암시적으로 사용됨.
- 따라서 명시할 필요 X
@JinUng41
refactor/#373 모임 내 약속 목록 조회 메서드 RxMoya로 변경
d057715
@JinUng41
chore/#373 불필요한 프로퍼티 삭제, 네이밍 변경, 세그먼트 컨트롤 설정
4335495
@JinUng41
chore/#373 빈 줄 삭제
9030dba
@JinUng41
refactor/#373 모임 전체 약속과 내가 참여한 약속 조회 메서드 분리
302bb0d
@JinUng41
refactor/#373 세그먼트가 변경될 때마다 네트워크 요청을 새로 하도록 수정
741a275
@JinUng41
fix/#373 약속 상세 화면으로부터 되돌아왔을 때, 이미지가 확대되는 현상 수정
c7848c4
@JinUng41 JinUng41 added ♻️ refactor 기존 코드를 리팩토링하거나 수정하는 등 사용 (생산적인 경우) 💙 JinUng 걸스 토크에 미쳐보고 싶다면 labels Sep 9, 2024
@JinUng41 JinUng41 self-assigned this Sep 9, 2024
@JinUng41 JinUng41 linked an issue Sep 9, 2024 that may be closed by this pull request
[refactor] 모임 내 약속 목록 기능 수정 및 QA 반영 #373
Closed
2 tasks
youz2me
youz2me approved these changes Sep 9, 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.

훨씬 깔끔해졌네요 !! 고생하셨습니다 🔥

hooni0918
hooni0918 approved these changes Sep 9, 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.

Lgtm

mmaybei
mmaybei approved these changes Sep 10, 2024
View reviewed changes
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.

수고하셨습니다!

@JinUng41 JinUng41 merged commit a0011ec into suyeon Sep 10, 2024
@JinUng41 JinUng41 deleted the refactor/#373-meeting-promise-list branch September 10, 2024 08:49
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

@hooni0918 hooni0918 hooni0918 approved these changes

@mmaybei mmaybei mmaybei approved these changes

Assignees

@JinUng41 JinUng41

Labels
💙 JinUng 걸스 토크에 미쳐보고 싶다면 ♻️ refactor 기존 코드를 리팩토링하거나 수정하는 등 사용 (생산적인 경우)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[refactor] 모임 내 약속 목록 기능 수정 및 QA 반영
4 participants
@JinUng41 @youz2me @hooni0918 @mmaybei