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] #86 - 랭킹에서 다른 사람 스탬프 상세 조회 연결, userId 분기처리 #91

Merged
merged 1 commit into from
Jan 8, 2023

Conversation

yangsubinn
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

  • 랭킹뷰에서 다른 사람 스탬프 상세 조회 연결
  • MissionList에서 ListDetail로 넘어갈때, 랭킹에서 넘어오는 경우 userId 전달
  • UseCase에서 userId 유무에 따라 구분
  • CustomNavigationBar에 hideRightButton 메소드 추가

📌 참고 사항

MissionList에서 ListDetail로 넘어갈때 userId가 필요한데,
그래서 MissionListViewModel 안에 otherUserId를 넣어서 내가 아닌 다른 유저 조회인 경우 사용할 수 있도록 해뒀습니다

스크린샷 2023-01-08 오후 3 01 48

이 부분은 엄 MissionListViewModel 안에 되도록 바로 데이터를 저장하지 않으려구 작업하신 것 같은데, 일단 이슈 먼저 해결하려구 public으로 otherUserId를 추가해둔거라 추후에 수정이 필요하다 느껴지면 수정해주셔두 될 것 같습니다. 🙇‍♀️

📸 스크린샷

기능 스크린샷
랭킹에서 글 상세 메인에서 글 상세 ezgif com-gif-maker (97)

📮 관련 이슈

Copy link
Contributor

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

수고하셨슴다!!

@yangsubinn yangsubinn merged commit 2e6962a into sopt-makers:develop Jan 8, 2023
@devxsby
Copy link
Member

devxsby commented Jan 9, 2023

고생하셨습니다 ❕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 문제 해결, 코드 수정 size/M 양수🌷
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 다른 사람 스탬프 확인할 경우 분기처리
3 participants