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: 탈퇴 기능을 구현합니다. #293

Merged
merged 5 commits into from
Aug 29, 2024
Merged

feat: 탈퇴 기능을 구현합니다. #293

merged 5 commits into from
Aug 29, 2024

Conversation

summermong
Copy link
Collaborator

🤔 어떤 문제가 발생했나요?

  • 탈퇴 기능(ㅠㅠ)이 추가되었습니다.

🎉 어떻게 해결했나요?

  • 탈퇴 사유를 post 요청으로 보내고 성공했을 때 delete 요청으로 실제 탈퇴까지 이뤄지도록 mutation을 사용했습니다.

📷 이미지 첨부 (Option)

2024-08-29.10.44.38.mov

⚠️ 유의할 점! (Option)

@summermong summermong added the feat # Add feature label Aug 29, 2024
@king2dwellsang king2dwellsang added the Review Plz🙏 # Review is not yet complete label Aug 29, 2024
Comment on lines 1 to 6
export interface DeleteAccountProps {
status: number;
code: string;
message: string;
data: object;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

음 개인적으로 응답인데 Props라는 네이밍보다는 Response가 좋아보이고, 현민님께서 작성해주신 Response type을 활용하면 더 깔끔할 것 같습니다!

@king2dwellsang king2dwellsang added Approved 🆗 # Review is completed and removed Review Plz🙏 # Review is not yet complete labels Aug 29, 2024
@summermong summermong merged commit d717edc into main Aug 29, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved 🆗 # Review is completed feat # Add feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants