Feat: 회원 탈퇴 API 추가 #32
Merged
+113
−20
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 tasks
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
전반적으로 회원 탈퇴 기능이 잘 구현되었습니다. 스터디장인 경우 탈퇴를 막는 로직과 인증 쿠키를 삭제하는 처리가 포함된 점이 좋습니다.
코드의 유지보수성과 일관성을 높이기 위해 몇 가지 개선점을 제안합니다. 주요 내용은 다음과 같습니다:
- 비즈니스 로직을 서비스 레이어로 분리하여 뷰의 역할을 명확히 하는 것을 제안합니다.
- API 응답 형식을 스웨거(OpenAPI) 문서와 일치시키는 것이 좋습니다.
- 하드코딩된 문자열을 상수로 정의하여 가독성과 유지보수성을 향상시키는 방안을 제안합니다.
자세한 내용은 각 파일의 코드 리뷰 코멘트를 참고해주세요.
- 쿠키 이름 상수화 - 스터디장이 회원 탈퇴할 때 발생하는 에러의 리스폰스에 스터디 목록 추가
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 개요 (Overview)
[ resolves #31 ]
🔎 변경사항 (Changes)
api/user/meDELETE -> 서비스 회원 탈퇴 API 추가