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/get pray list sort #174

Merged
merged 3 commits into from
May 10, 2024
Merged

Fix/get pray list sort #174

merged 3 commits into from
May 10, 2024

Conversation

baebae02
Copy link
Member

@baebae02 baebae02 commented Apr 26, 2024

✅ PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 변경
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

📌 관련 이슈

✨ 과제 내용

  1. 메인화면에서 기도를 하면 어쩔 때는 그 순서 그대로 있고, 어쩔 때는 맨 밑으로 내려가고, 이렇게 랜덤으로 순서가 배치되는데 계속 바뀌지 않도록 created_at으로 asc 정렬이 필요할 것 같습니다

grouppray get 에서 날짜 정렬이 이상한 것 같습니다 🤔 최신의 것이 위로 가도록 정렬이 필요한 것 같습니다.
=> Tree Map을 사용하여 그룹화 된 기도제목들이 key를 기준으로 역정렬 되도록 추가하였습니다.

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

@baebae02 baebae02 added the ✅해결 각종 오류 및 문제를 해결했을 때 label Apr 26, 2024
@baebae02 baebae02 self-assigned this Apr 26, 2024
Copy link
Contributor

@dong2ast dong2ast left a comment

Choose a reason for hiding this comment

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

원인을 잘 찾아서 해결하셨네요 고생하셨습니다 👍 👍

Copy link
Member

@Sion99 Sion99 left a comment

Choose a reason for hiding this comment

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

TreeMap은 처음인데 알아갑니다! 고생하셨습니다 ㅎㅎ

@baebae02 baebae02 merged commit c413f61 into main May 10, 2024
1 check passed
@baebae02 baebae02 deleted the fix/get_pray_list_sort branch May 10, 2024 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅해결 각종 오류 및 문제를 해결했을 때
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants