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

[홈] 캡슐 순서를 랜덤에서 고정 순서로 변경 #119

Merged
merged 4 commits into from
Dec 13, 2022

Conversation

solbat
Copy link
Member

@solbat solbat commented Dec 13, 2022

제출 전 필수 확인 사항:

  • Merge 하는 브랜치가 올바른가?
  • 코드 컨벤션을 준수하는가?
  • 빌드가 되는 코드인가요?
  • 버그가 발생하지 않는지 충분히 테스트 해봤나요?

작업 내용:

  • HomeViewModel : 랜덤으로 출력하지 않고 고정 순서로 출력하도록 변경
  • HomeVC : 홈 화면에 돌아갈 때마다 매번 fetchCapsules하여 refresh 하도록
  • CapsuleType의 title의 순서와 표현 변경

이번에 공들였던 부분:

  • 홈 화면에 보이는 캡슐들의 순서를 고정하는 것이 좋다는 의견이 있었습니다.
  • 따라서 캡슐들의 순서를 고정하도록 하였고, 우선 제 판단으로 순서를 정해보았습니다.
  • 순서가 마음에 들지 않으면 함께 논의하여 순서를 조정하면 좋을 것 같습니다.
  • 또한 캡슐 클릭하여 CapsuleAccess에 들어갔다가 그냥 나왔을 경우 refresh를 진행합니다.
  • 따라서 실시간 정보에 가깝게 보여지도록 구현하였습니다.

@solbat solbat self-assigned this Dec 13, 2022
@solbat solbat changed the title Bugfix home [홈] 홈 화면 캡슐 순서를 랜덤에서 고정 순서로 변경 Dec 13, 2022
@solbat solbat changed the title [홈] 홈 화면 캡슐 순서를 랜덤에서 고정 순서로 변경 [홈] 캡슐 순서를 랜덤에서 고정 순서로 변경 Dec 13, 2022
Copy link
Member

@trumanfromkorea trumanfromkorea left a comment

Choose a reason for hiding this comment

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

완벽한데요?;; 바로 Merge 하시죠?!!;;;

@solbat solbat merged commit 044a5ac into develop Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants