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

[Quest] Main Quest List 로직 수정 #35

Merged
merged 13 commits into from
Nov 1, 2020
Merged

[Quest] Main Quest List 로직 수정 #35

merged 13 commits into from
Nov 1, 2020

Conversation

chelwoong
Copy link
Member

@chelwoong chelwoong commented Nov 1, 2020

Close #34

  • ViewModel에서 section을 결정하지 않고 주어진 대로 관리만 하도록 변경
  • 완료 탭에서 상시, 학습 퀘스트가 섞여 나올 수 있도록 section을 하나로 통합 -> 대신 QuestList에서 state에 따라 sort를 해서 처리

- quest의 순서는 이전에 정해주고 단순히 주어진 대로 관리하도록 수정
- 가독성을 위해 extension으로 분리
- 높이는 viewModel에서 관리해주도록 수정
- quest의 순서는 이전에 정해주고 단순히 주어진 대로 관리하도록 수정
- 가독성을 위해 extension으로 분리
- 높이는 viewModel에서 관리해주도록 수정
@chelwoong chelwoong added the FE Front-end issue label Nov 1, 2020
@chelwoong chelwoong self-assigned this Nov 1, 2020
@chelwoong chelwoong merged commit 3c2ffb4 into Yapp-17th:feature/quest Nov 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE Front-end issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant