Skip to content

[#233] 회원 탈퇴 구현 구현#234

Merged
halionaz merged 14 commits intodevfrom
feat/#233/delete-user
Sep 23, 2025
Merged

[#233] 회원 탈퇴 구현 구현#234
halionaz merged 14 commits intodevfrom
feat/#233/delete-user

Conversation

@halionaz
Copy link
Copy Markdown
Member

❗ 연관 이슈

📌 내용

  • 제곧내
  • 그리고 배포서버에서 로그인 url에 붙어있는 redirect url 파람 없앴음

@halionaz halionaz requested a review from Copilot September 23, 2025 12:02
@halionaz halionaz self-assigned this Sep 23, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 23, 2025

Deploy Preview for repicka ready!

Name Link
🔨 Latest commit d8e92ae
🔍 Latest deploy log https://app.netlify.com/projects/repicka/deploys/68d28d16d97fb90008db0141
😎 Deploy Preview https://deploy-preview-234--repicka.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Implements user account deletion functionality and removes redirect URL parameters from login URLs for deployment server. This adds a new delete account feature while simplifying the OAuth login flow.

  • Added user account deletion with confirmation dialog
  • Updated login page to conditionally include redirect URL parameters
  • Refactored Menu component logic for better readability

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/pages/LoginPage/index.tsx Updates OAuth login URLs to conditionally include redirect parameters
src/features/my/components/MyTrade/index.tsx Refactors Menu component and adds DeleteAccountButton
src/features/my/components/MyTrade/DeleteAccountButton.tsx New component for account deletion with confirmation dialog
src/features/my/apis/useDeleteUser.ts API hook for user deletion with query cache invalidation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/features/my/components/MyTrade/DeleteAccountButton.tsx
Comment thread src/features/my/apis/useDeleteUser.ts
@halionaz halionaz merged commit 5de28b0 into dev Sep 23, 2025
4 checks passed
@halionaz halionaz deleted the feat/#233/delete-user branch September 23, 2025 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

탈퇴 추가

3 participants