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] 게시글 조회시 각종 이미지 조회 추가 #119 #136

Merged
merged 11 commits into from
Jan 23, 2025

Conversation

yschoi123
Copy link
Collaborator

@yschoi123 yschoi123 commented Jan 23, 2025

🚀 Pull Request

게시글 조회시 매핑 이미지 및 작성자 프로필 이미지 조회 추가

#️⃣ 연관된 이슈

📋 작업 내용

  • 게시글 수정,삭제,등록 컨트롤러 메서드명을 더 구체적으로 수정
  • 조회수 증가 로직을 서비스단에서 분리하여 컨트롤러에서 각각 호출하도록 변경
  • FileService에 게시글 대표 이미지 조회하는 서비스 메서드 추가
  • 게시글 조회 서비스에 작성자의 프로필 이미지와 게시글에 연관된 이미지를 조회하는 로직 추가
  • 게시글 엔티티(Post)에 작성자 id를 읽어오는 메서드 추가

🔧 변경된 코드 설명

.

✅ 테스트 여부

  • 테스트 코드 실행 여부
  • 서버 실행 여부
  • 스웨거 테스트 여부

👽 비고

.

@yschoi123 yschoi123 self-assigned this Jan 23, 2025
Copy link
Collaborator

@yr0202 yr0202 left a comment

Choose a reason for hiding this comment

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

저도... 댓글에 이미지처리를 빼먹었습니다...ㅜ 빨리 추가할게요...

@yschoi123 yschoi123 merged commit d58f823 into dev Jan 23, 2025
2 checks 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.

3 participants