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: Post 저장 기능 구현 #55

Conversation

Shsin9797
Copy link
Contributor

🔎 작업 내용

Post save 기능을 구현했습니다.

💬 리뷰 요구사항 (선택)

  1. response 값에 어떤걸 넣어주는게 포멀 한가요?
  • 게시글의 경우, 게시글 id 와 게시글 제목 정도만 반환해도 되나요? 아니면 게시글 내용도 반환하는게 좋나요 ?
  • 생성일자 안넣어도 되나요?
  1. document 보고+ gpt 내용 참고해서 ResponseEntity 에 새로생성된 게시글의 url 을 헤더에 넣어주는 방식으로 구현했는데, url fomat 생성하는 부분이 코드변경에 취약할 것 같아 보입니다. 이부분을 개선할 방법이 있나요? 아니면 이렇게 코드 작성하는게 잘못되었는지 궁금합니다.

📸 이미지 첨부 (선택)

  1. 현재는 게시글 id랑 title 정도만 반환하고 있습니다.
    image

  2. 이부분에서 v1, v2 가 추가된다거나 하는 url 이 변경된경우 , 일일이 찾아가서 수정하기 힘들어보입니다.
    image

➕ 이슈 링크

@Shsin9797
Copy link
Contributor Author

Shsin9797 commented Oct 10, 2024

🔎 작업 내용

Post save 기능을 구현했습니다.
400,500 처리는 할예정 입니니다.

💬 리뷰 요구사항 (선택)

  1. response 값에 어떤걸 넣어주는게 포멀 한가요?
  • 게시글의 경우, 게시글 id 와 게시글 제목 정도만 반환해도 되나요? 아니면 게시글 내용도 반환하는게 좋나요 ?
  • 생성일시 안넣어도 되나요? ( api 명세서에는 처음엔 생성일시 json 으로 반환하는걸로 작성 했었어요.. 굳이 반 안해도 되는지 궁금합니다. )
    image
  1. document 보고+ gpt 내용 참고해서 ResponseEntity 에 새로생성된 게시글의 url 을 헤더에 넣어주는 방식으로 구현했는데, url fomat 생성하는 부분이 코드변경에 취약할 것 같아 보입니다. 이부분을 개선할 방법이 있나요? 아니면 이렇게 코드 작성하는게 잘못되었는지 궁금합니다.

📸 이미지 첨부 (선택)

  1. 현재는 게시글 id랑 title 정도만 반환하고 있습니다.
    image
  2. 이부분에서 v1, v2 가 추가된다거나 하는 url 이 변경된경우 , 일일이 찾아가서 수정하기 힘들어보입니다.
    image

➕ 이슈 링크

@Shsin9797 Shsin9797 closed this Oct 11, 2024
@Shsin9797
Copy link
Contributor Author

위치를 잘못올린거같아서 close 했습니다!

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.

1 participant