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

[NB-48, NB-155] 메인 화면에서의 게시글 조회 구현 #31

Merged
merged 4 commits into from
Jul 31, 2024

Conversation

liljoon
Copy link
Contributor

@liljoon liljoon commented Jul 31, 2024

개요

NB-48:

  • 랜덤한 추천 게시글 조회 api 구현
  • seed 설정을 통해 일관성 있는 페이징 구현

NB-155:

  • 팔로우한 대상의 최근 3일 게시글 구현
  • 페이징 처리

기타:

  • 순환 참조 방지를 위한 Mapper 구조 변경
  • Mapper에서 다른 객체를 호출하지 않고 상위에서 파라미터를 통해 전달받는 구조

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 코드 리팩토링

PR Checklist

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

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고 (Ctrl + 클릭하세요.)
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

liljoon added 4 commits July 30, 2024 11:50
- 최근 3일을 기준으로 팔로우한 유저들의 최근 게시글 조회
- PostSimpleResponseDto 에 날짜 표기
- 자신 게시물 제외 랜덤 5개 페이징하여 조회
- CommentResponse 와 함께 반환
- 순환 참조 방지를 위한 Mapper에서 다른 Service 제거
@liljoon liljoon requested a review from Aram-su July 31, 2024 03:38
Copy link
Contributor

@Aram-su Aram-su left a comment

Choose a reason for hiding this comment

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

코드에 문제 없는 것 같습니다

@liljoon liljoon merged commit 116913b into dev Jul 31, 2024
1 check passed
@liljoon liljoon deleted the feat/main-posts branch July 31, 2024 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants