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

12주차 과제 완료✨ #24

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

12주차 과제 완료✨ #24

wants to merge 3 commits into from

Conversation

tigowler
Copy link
Member

📌 관련 이슈

✨ 과제 내용

1. 댓글 작성

  • 💡 /<int:blog_id>/comment
  • 댓글이 작성될 블로그 게시글 pk를 명시하도록 url을 설정함

2. 댓글 수정, 삭제

  • 💡 /comment/<int:comment_id>
  • 수정할 댓글 pk를 명시하여 url 설정

3. 댓글 조회

  • 💡 /<int:blog_id>/comment
  • 특정 게시글에 달린 댓글 목록을 가져오는 것이 필요하다고 생각하여, 블로그 게시글 pk를 명시하도록 url 설정

📸 스크린샷(선택)

1. 댓글 작성

post

2. 댓글 수정

put

3. 댓글 삭제

delete

4. 댓글 조회

get

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

앞서 blog 게시글에 대한 CRUD를 구현할 때는 path variable이 있을 때와 없을 때로 url과 기능을 구분했는데, 댓글을 구현하다보니 blog와 comment 모델에 대한 pk가 둘 다 path variable로 필요한 경우가 있더라구요..!

url
심지어 둘다 int라 우선 path variable의 위치를 바꿔서 기능하도록 했는데, 보통.. 이렇게 해도 되나요..?? 혹시 더 나은 다른 방법이 있는지 궁금합니다!

@tigowler tigowler requested a review from a team July 20, 2022 16:27
@tigowler tigowler self-assigned this Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 멋사 12차 과제🦁
1 participant