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

[BE-REFACTOR] 바이옴 포켓몬 티어 정렬 #363

Open
wants to merge 2 commits into
base: be/develop
Choose a base branch
from

Conversation

jongmee
Copy link
Contributor

@jongmee jongmee commented Oct 13, 2024

🍄 PR 확인 사항

PR이 다음 요구 사항을 충족하는지 확인하세요. :

  • API 명세서가 업데이트 혹은 작성이 되어 있나요? -> 머지되고 작성할래요

현재 작업은 어떤 이슈를 해결한 것인지 설명해주세요.

Issue Number: #362

  • 보스 포켓몬 희귀도 높은 순으로 티어 정렬
  • 요구사항에 관계 없이 동적으로 쿼리스트링을 사용해서 정렬

기존 코드에서 변경된 점이 있다면 설명해주세요. (추가 X)

  • 있음
  • 없음

이건 제안인데요! 쿼리 스트링으로 보스 포켓몬과 야생 포켓몬들 티어 정렬 기준을 받도록 api를 수정하면 어떨까요? 사용자 활동 로그를 바탕으로 정렬 순서도 고치기로 했던 걸로 기억하는데 정렬 같은 조건문은 요구사항에 따라 바뀌는 것보다 이렇게 설계하는게 더 유연해보여서 제안합니다 :)

보스 포켓몬들에 대해 티어 높은 순으로 정렬만 요구사항이었기 때문에 쿼리스트링 default 값을 보스 포켓몬은 desc, 야생 포켓몬은 asc로 설정해보았습니다. (하위 호환성 O)

별로라면 철회하겠읍니다..

@jongmee jongmee added the BE_REFACTOR 🌿 백엔드 리팩토링 label Oct 13, 2024
@jongmee jongmee self-assigned this Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE_REFACTOR 🌿 백엔드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant