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(be): add guards, question update api and question delete api #87

Merged
merged 9 commits into from
Nov 14, 2024

Conversation

yu-yj215
Copy link
Collaborator

@yu-yj215 yu-yj215 commented Nov 14, 2024

개요

  • 세션과 토큰의 유효성을 검사하는 가드, 질문이 존재하는지 검사하는 가드, 질문의 주인인지를 검사하는 가드를 만들었습니다.

  • Patch와 delete를 이용한 질문의 상태변경을 만들었습니다.

  • 답변완료여부를 통해 수정과 삭제의 권한을 제어합니다.

  • 답변완료는 요청으로 false로 만들 수 없습니다.

  • 질문의 생성자인지를 판단해서 pinned와 closed의 권한을 제어합니다.

  • 경로 별칭을 만들었습니다.

이슈

@yu-yj215 yu-yj215 added Feature New feature or request Refactor labels Nov 14, 2024
@yu-yj215 yu-yj215 self-assigned this Nov 14, 2024
@yu-yj215 yu-yj215 requested review from shl0501, cjeongmin and wlgh1553 and removed request for shl0501 and cjeongmin November 14, 2024 12:21
Copy link
Collaborator

@cjeongmin cjeongmin left a comment

Choose a reason for hiding this comment

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

LGTM ✓

Copy link
Collaborator

@wlgh1553 wlgh1553 left a comment

Choose a reason for hiding this comment

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

LGTM!!! 고생하셨습니다!

@yu-yj215 yu-yj215 merged commit 4ef5ce9 into main Nov 14, 2024
1 check passed
@wlgh1553 wlgh1553 deleted the feat(be)/questions branch November 15, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request Refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat]: 답변 완료가 되지 않은 질문 수정, 삭제 기능
3 participants