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.

Sign up for GitHub

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

Jump to bottom

질문 삭제 기능 추가 및 API 요청 핸들러 수정 #306

Merged
merged 4 commits into from
May 9, 2024
Merged

질문 삭제 기능 추가 및 API 요청 핸들러 수정 #306

merged 4 commits into from
May 9, 2024

Conversation

JayJayleee
Copy link
Contributor

@JayJayleee JayJayleee commented May 9, 2024 •
edited
Loading

✨(#이슈 번호)

#285

✏️내용

  • 질문 삭제 기능 추가
  • 질문 삭제 시, 답변 있을 경우 에러 처리
  • API 핸들러 에러 코드 리턴하도록 수정

📸스크린샷

🎁참고사항

  • 답변 삭제 API가 분리되었을까요? 기존에는 질문 삭제에 통합으로 있던 것으로 기억하는데, 확인 부탁드립니다!(현재는 답변 삭제 시 에러 토스트만 발생)
  • 질문 수정도 답변이 있을 떄는 409 에러가 발생하는 것 같습니다. 애초에 수정 페이지로 못가도록, 에러 처리가 필요해 보입니다.

@JayJayleee JayJayleee self-assigned this May 9, 2024
@JayJayleee JayJayleee merged commit 5dec22b into Team-Coverflow:develop May 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kmin-jeong kmin-jeong Awaiting requested review from kmin-jeong

@HA-SEUNG-JEONG HA-SEUNG-JEONG Awaiting requested review from HA-SEUNG-JEONG

@fakerdeft fakerdeft Awaiting requested review from fakerdeft

@raxchaz raxchaz Awaiting requested review from raxchaz

Assignees

@JayJayleee JayJayleee

Labels
BugFix
Projects
[2024] Coverflow
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JayJayleee