Skip to content

Conversation

@seongjunnoh
Copy link
Collaborator

📝 요약

resolved : #447

🔖 변경 사항

  1. 게시글 상세조회 api ("/space/{spaceId}/board/{boardId}/post/{postId}") 의 response에 요청 보내는 유저가 해당 게시글의 작성자인지를 알려주는 정보 추가

  2. 게시글 리스트 조회 api ("/space/{spaceId}/board/{boardId}/post") 의 response에 요청 보내는 유저가 해당 게시글의 작성자인지를 알려주는 정보 추가

✅ 리뷰 요구사항

<로컬 테스트 해보다가 알게된 사실>
현재 S3 upload 시에 업로드 할 수있는 파일 확장자를 AllowedImageFileExtensions.java enum 파일에 명시해 둔 상황
image

그런데 제가 HEIC 이미지 파일을 첨부파일로 해서 게시글 생성 요청을 보내니, 디스코드에 제대로 보이지 않습니다
image

-> 이런식으로 미리보기가 안됨 (링크 클릭하면 사진이 현재 디바이스에 다운로드 되고, 저장된 사진을 클릭하면 보이긴 함)
-> 위 사진은 jpeg 파일임 -> 미리보기 잘됨

개인적인 추측으로는 heic 파일이 용량이 커서 그런가? 싶긴한데 정확한 이유를 잘 모르겠네여. (구글링 해봐도 잘 안나오네여)
혹시 관련해서 아는 내용 있으시면 리뷰 남겨주시면 감사하겠습니다

📸 확인 방법 (선택)

image -> 변경사항 1 로컬 테스트 결과 image -> 변경사항 2 로컬 테스트 결과

로컬로는 전부 제가 글 생성하고 제가 확인하는 거여서 isPostOwner 값이 전부 true긴 합니다. dev로 배포 후 한번 확인이 필요하긴 합니다.



📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@seongjunnoh seongjunnoh linked an issue Apr 15, 2025 that may be closed by this pull request
Copy link
Member

@hyunn522 hyunn522 left a comment

Choose a reason for hiding this comment

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

LGTM

@seongjunnoh seongjunnoh merged commit 5778e0f into develop Apr 16, 2025
2 checks passed
@seongjunnoh seongjunnoh deleted the refactor/#447/게시글-상세-조회-response-추가 branch April 16, 2025 09:08
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.

Refactor : 게시글 api response 정보 추가

3 participants