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#43 게시물 전체 조회 기능 구현 #44

Merged
merged 3 commits into from
Nov 12, 2024
Merged

feat#43 게시물 전체 조회 기능 구현 #44

merged 3 commits into from
Nov 12, 2024

Conversation

1winhyun
Copy link
Contributor

1. 무슨 이유로 코드를 변경했나요?


게시물 전체 조회 기능 구현

2. 어떤 위험이나 장애를 발견했나요?


3. 관련 스크린샷을 첨부해주세요.


4. 완료 사항


5. 추가 사항

@1winhyun 1winhyun requested review from jj0526 and soyesenna November 11, 2024 05:54
@1winhyun 1winhyun self-assigned this Nov 11, 2024
@1winhyun 1winhyun linked an issue Nov 11, 2024 that may be closed by this pull request
@@ -99,6 +99,16 @@ public PostResponseDTO getPost(Long postId){
return toPostResponseDTO(post);
}

public List<PostResponseDTO> getAllPosts(){
Copy link
Contributor

Choose a reason for hiding this comment

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

전체 게시물 말고 팔로우한 사람들의 글들을 가져와야될거 같습니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

팔로우한 사람들의 글만 가져오게 설정하였습니다.

Copy link
Collaborator

@soyesenna soyesenna left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

@1winhyun 1winhyun merged commit bc80477 into main Nov 12, 2024
1 check passed
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.

[feat] #43 게시물 전체 조회 구현
3 participants