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] Qna 검색 api 개발 및 댓글 등록 api 구현 #256

Open
wants to merge 5 commits into
base: be-dev
Choose a base branch
from

Conversation

ashd89
Copy link

@ashd89 ashd89 commented Sep 23, 2024

연관 이슈

close #254 #255

작업 내용

답변의 댓글을 superComment를 활용하여 등록 가능
답변 및 상위 댓글의 존재에 관련 에러 처리
답변에 대해 댓글을 등록 가능
댓글에 관한 에러 처리

스크린샷 (선택)

리뷰 요구사항 혹은 기타 (선택)

그 원래는 댓글 등록 브랜치를 만들었었는데... 날아가 버렸네요 하하하

@ashd89 ashd89 added 🍃 backend 백엔드 작업 ✏️ feature 추가 labels Sep 23, 2024
@ashd89 ashd89 self-assigned this Sep 23, 2024
Copy link
Contributor

@sue06004 sue06004 left a comment

Choose a reason for hiding this comment

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

최대한 pr 나눠서 올려줘..


//qna 답변 좋아요
@PostMapping("/like")
public BaseResponse<Long> checkAnsLike(@RequestParam Long qnaBoardId, @RequestParam Long answerId, @AuthenticationPrincipal CustomUserDetails customUserDetails) {
Copy link
Contributor

Choose a reason for hiding this comment

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

RequestParam 그대로인데?

@GetMapping("/qna-like")
public BaseResponse<Long> checkQnaLike(Long qnaBoardId, @AuthenticationPrincipal CustomUserDetails customUserDetails) {
@PostMapping("/like")
public BaseResponse<Long> checkQnaLike(@RequestParam Long qnaBoardId, @AuthenticationPrincipal CustomUserDetails customUserDetails) {
Copy link
Contributor

Choose a reason for hiding this comment

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

여기도

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍃 backend 백엔드 작업 ✏️ feature 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants