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.

Sign up for GitHub

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

Jump to bottom

fix: 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 #438

Merged
merged 2 commits into from
Jun 21, 2023
Merged

fix: 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 #438

merged 2 commits into from
Jun 21, 2023

Conversation

nyj001012
Copy link
Contributor

@nyj001012 nyj001012 commented Apr 12, 2023 •
edited
Loading

개요

  • 전체 대출반납 기록 조회 시, 정렬 순서가 최신 순이 아니라 createdAt, login의 내림차순임
  • 따라서 대출, 반납 등의 수정이 일어났을 때를 기준으로 하여 내림차순으로 정렬함
  • 반납 시, lending.updatedAt이 갱신되지 않는 점 수정

작업 사항 및 변경점

  • 함수 getHistoriesItems 정렬 조건을 updatedAt: 'DESC'로 변경
  • 함수 updateLending 호출 시, updatedAt 값도 같이 업데이트

목적

대출/반납 기록 조회 시, 최신 순으로 정렬하기 위함

@nyj001012
bug(history): 정렬 조건을 수정 시간 내림차순으로 변경
665b01f 
기록 조회 시, 정렬 조건을 수정 시간 내림차순으로 변경
@nyj001012
bug(history): 반납 시, updatedAt 값 갱신하도록 수정
ca1077a 
반납 시, updatedAt의 값도 갱신하도록 수정
@nyj001012 nyj001012 added the bug Something isn't working label Apr 12, 2023
@nyj001012 nyj001012 self-assigned this Apr 12, 2023
@nyj001012 nyj001012 linked an issue Apr 12, 2023 that may be closed by this pull request
전체 대출/반납 기록 조회 시, 정렬 순서가 최신 순이 아님 #436
Closed
@nyj001012 nyj001012 changed the title bug: 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 436 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 Apr 14, 2023
@nyj001012 nyj001012 changed the title 436 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 bug(lending): 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 Apr 14, 2023
@nyj001012 nyj001012 changed the title bug(lending): 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 bug: 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 Apr 14, 2023
jimin52
jimin52 approved these changes Jun 14, 2023
View reviewed changes
Copy link
Member

@jimin52 jimin52 left a comment

Choose a reason for hiding this comment

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

createdAt 대신 updatedAt 으로 정렬하게 수정한 것 확인했습니다.

@scarf005 scarf005 changed the title bug: 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 fix: 전체 대출반납 기록 조회 시 정렬 순서가 최신 순이 아님 Jun 20, 2023
@nyj001012 nyj001012 merged commit a3dd80f into develop Jun 21, 2023
@nyj001012 nyj001012 deleted the 436-전체-대출반납-기록-조회-시-정렬-순서가-최신-순이-아님 branch June 21, 2023 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jimin52 jimin52 jimin52 approved these changes

@scarf005 scarf005 scarf005 approved these changes

Assignees

@nyj001012 nyj001012

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

전체 대출/반납 기록 조회 시, 정렬 순서가 최신 순이 아님
3 participants
@nyj001012 @jimin52 @scarf005