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/3-scrapPost] 쥬시글 스크랩/취소 API #16

Merged
merged 1 commit into from
Sep 18, 2023

Conversation

JoongHyun-Kim
Copy link
Member

📌 이슈 번호

#3

📢 의논 사항

  • none

@JoongHyun-Kim JoongHyun-Kim added the ✨ feature New feature label Sep 18, 2023
@JoongHyun-Kim JoongHyun-Kim self-assigned this Sep 18, 2023
@Haeun-Y Haeun-Y merged commit 0a39431 into develop Sep 18, 2023
* [POST] 스크랩/취소
*/
@ResponseBody
@GetMapping("/{postIdx}")
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분 주석에는 POST 메소드인데 주석은 @GetMapping을 사용한 걸 보니까 GET 메소드로 잘못 적은 것 같습니다. GET 메소드 사용하면 쥬시글 상세조회랑 요청 URL이 똑같아요. 수정 부탁드립니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants