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

[FEATURE] 마감임박순 정렬 구현 #214

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

shindong96
Copy link
Member

작업 내용

  • 쿼리dsl을 이용해 마감임박순 정렬을 구현한다.
  • 기존에 OrderSpecifier을 Array로 응답하여 우선순위대로 적용시킴

논의점

지금 Service에서 LocalDateTime.now()를 넘기는데 이 부분이 DB에 저장된 schedule과 시간대가 맞지 않을 경우 문제가 발생할 수 있습니다. 이 부분에 대해 논의가 필요합니다.

resolve #213

@shindong96 shindong96 added the ⚡️ feature new feature label Jul 31, 2024
@shindong96 shindong96 requested a review from kyukong July 31, 2024 07:09
@shindong96 shindong96 self-assigned this Jul 31, 2024
Copy link
Member

@kyukong kyukong left a comment

Choose a reason for hiding this comment

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

안녕하세요
이전에 마감시간 관련 QueryDsl 찾아봤을 때 찾기 쉽지 않았었는데 고생많으셨습니다 😅

제 기억에 LocalDateTime 은 OS 에 설정된 Timezone 에 영향을 받았었습니다.
그래서 DB 서버와 설정이 달라서 문제가 발생했던 적이 있었네요.
각 서버마다 Timezone 을 일치시키는 것도 좋은 방법일것 같고,
LocalDateTime 에 Timezone 을 추가한 ZonedDateTime 을 사용하는 것도 방법일듯 합니다!!

수고 많으셨습니다!

@shindong96 shindong96 merged commit 50e76fb into develop Aug 7, 2024
1 check passed
@shindong96 shindong96 deleted the feature/#213/sort-by-end-date branch August 7, 2024 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚡️ feature new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] 마감임박 순 정렬 기능
2 participants