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

[feat #42] 스크랩 목록 조회 API #50

Merged
merged 12 commits into from
Aug 17, 2024
Merged

Conversation

dudxo
Copy link
Collaborator

@dudxo dudxo commented Aug 16, 2024

관련 이슈

📑 작업 상세 내용

  • 스크랩 목록 조회 API
    • QueryDSL 스크랩 목록 조회 작성
    • Repository/Controller Test 작성
  • MemberRepository 테스트 내 검증 시 하드 코딩 부분 수정

💫 작업 요약

  • 스크랩 목록 조회 API

🔍 중점적으로 리뷰 할 부분

  • 스크랩 목록 조회 시 게시물 업데이트 기준으로 가져왔는데, 스크랩 기준으로 할까요?
  • 내가 쓴 글 전체 조회 API와 응답 DTO가 동일한데, 재사용하려다가 따로 나눴습니다.
    • 스크랩 목록 조회 응답 DTO 필드로 더 필요한게 있는지 확인 부탁드립니다!

@dudxo dudxo added the ✨ feat 기능 추가 label Aug 16, 2024
@dudxo dudxo self-assigned this Aug 16, 2024
@dudxo dudxo linked an issue Aug 16, 2024 that may be closed by this pull request
1 task
Copy link

github-actions bot commented Aug 16, 2024

Test Results

74 tests   69 ✅  7s ⏱️
18 suites   5 💤
18 files     0 ❌

Results for commit 4d0c7c7.

♻️ This comment has been updated with latest results.

@hyun2371 hyun2371 self-requested a review August 17, 2024 03:27
@hyun2371
Copy link
Member

hyun2371 commented Aug 17, 2024

스크랩 목록 조회 시 게시물 업데이트 기준으로 가져왔는데, 스크랩 기준으로 할까요?

스크랩 순이면 스크랩 수 기준 내림차순 정렬 말씀하시는건가요?
저는 업데이트 기준이 좋은 것 같습니다.

근데 저는 개인적으로 createdAt을 기준으로 업데이트하는게 조회 성능이 더 좋게 나오지 않을까 하네요.
답글이 달리거나 질문글 내용이 수정될 때마다 updatedAt이 갱신되니까 정렬이 빈번히 바뀌잖아요.
그로 인해 캐싱이 어렵지 않을까 싶습니다..
사용자 입장에서도 조회할 때마다 순서가 매번 바뀌면 헷갈릴지 않을까 생각도 들고요

이 부분은 다음주 전체 회의 때 논의해봐도 좋을 것 같아요~

Copy link
Member

@hyun2371 hyun2371 left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다~

@dudxo
Copy link
Collaborator Author

dudxo commented Aug 17, 2024

스크랩 목록 조회 시 게시물 업데이트 기준으로 가져왔는데, 스크랩 기준으로 할까요?

스크랩 순이면 스크랩 수 기준 내림차순 정렬 말씀하시는건가요? 저는 업데이트 기준이 좋은 것 같습니다.

근데 저는 개인적으로 createdAt을 기준으로 업데이트하는게 조회 성능이 더 좋게 나오지 않을까 하네요. 답글이 달리거나 질문글 내용이 수정될 때마다 updatedAt이 갱신되니까 정렬이 빈번히 바뀌잖아요. 그로 인해 캐싱이 어렵지 않을까 싶습니다.. 사용자 입장에서도 조회할 때마다 순서가 매번 바뀌면 헷갈릴지 않을까 생각도 들고요

아뇨 스크랩 createdAt 순으로 정렬하는걸 말씀드렸습니다! createdAt를 업데이트한다는게 무슨 말씀인가요?

@hyun2371
Copy link
Member

.orderBy(qp.updatedAt.desc())
게시글을 updatedAt 기준으로 정렬하시는 것 같은데 , createdAt 기준으로 정렬하는게 조회 성능, 유저 편의성 측면에서 더 좋지 않을까 말씀드린거였습니다~~

@dudxo
Copy link
Collaborator Author

dudxo commented Aug 17, 2024

.orderBy(qp.updatedAt.desc()) 게시글을 updatedAt 기준으로 정렬하시는 것 같은데 , createdAt 기준으로 정렬하는게 조회 성능, 유저 편의성 측면에서 더 좋지 않을까 말씀드린거였습니다~~

저는 업데이트 일자로 보여주면 더 좋지 않을까 생각했었는데 가현님 의견이 더 좋은 것 같네요! 수정하겠습니다!

수정했습니다!

@dudxo dudxo merged commit c9bdf8c into dev Aug 17, 2024
3 checks passed
@dudxo dudxo deleted the feat/#42/member-savedQuestionPost branch August 17, 2024 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ 스크랩 목록 조회 API
2 participants