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/37 - 게시글 수정/삭제 API 구현 #42

Merged
merged 4 commits into from
May 26, 2024
Merged

Conversation

koojun99
Copy link
Contributor

@koojun99 koojun99 commented May 25, 2024

🚀 개요

게시글 수정/삭제 API 구현

🔍 변경사항

  • PostMapper.getPostDetails에서 jpa 쿼리를 보내지 않고, PostService에서 보내 게시글 이미지의 url을 Mapper로 보내도록 수정
  • 게시글 수정 API
  • 게시글 삭제 API

⏳ 작업 내용

  • 게시글 수정 API 구현
  • 게시글 삭제 API 구현

📝 논의사항

수정/삭제 시, 대상 게시글의 작성자와 현재 로그인 해 있는 사용자가 동일한지 확인하는 로직을 구현하고자 하는데, 하나의 User가 여러 트리하우스에 가입해 Member를 여러개 가진 경우, 이를 어떻게 처리해야할까요? 지난번 끔찍했던 코드(getTreeProfile)가 발생한 원인이 이곳입니다.

@koojun99 koojun99 added the ✨ feat New feature or request label May 25, 2024
@koojun99 koojun99 self-assigned this May 25, 2024
@koojun99 koojun99 linked an issue May 25, 2024 that may be closed by this pull request
2 tasks
@koojun99 koojun99 requested a review from CYY1007 May 26, 2024 07:29
@CYY1007 CYY1007 merged commit 3209d38 into develop May 26, 2024
HyoBN pushed a commit that referenced this pull request Aug 9, 2024
Feature/37 - 게시글 수정/삭제 API 구현
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚀 [feature] 게시글 수정/삭제 API 구현
2 participants