Skip to content

feat(user): User Information Delete Method - #464

Merged
zigzagdev merged 10 commits into
mainfrom
feat/user-information-delete
Jul 26, 2026
Merged

zigzagdev merged 10 commits into
mainfrom
feat/user-information-delete

Conversation

@zigzagdev

@zigzagdev zigzagdev commented Jul 26, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Add deleteUser(id) to the user API client (DELETE /api/v1/users/:id, 204 No Content — no response body, Promise<void>)
  • No mapper stage: deletion only ever needs the target id, there's no form-values object with a different shape to convert (see feat: 削除対象をAPIリクエストへ変換するmapperを追加 #424, closed as not needed)
  • Add useDeleteUser hook: manages done/isLoading/error, aborts the in-flight request on resubmit/unmount, matching useCreateUser/useUpdateUser
  • Wire delete into the existing /users/:id page instead of a separate route: UserGetContainer drives useDeleteUser and navigates to /heritages once deletion completes
  • UserProfileView gains a "Delete User" button that opens a confirmation dialog (MUI Dialog); confirming calls the delete, cancelling is a no-op; shows an error panel on failure
  • Add userDeleteButton/userDeleteConfirmTitle/userDeleteConfirmMessage/userDeleteError i18n keys (en/ja)

Related

Closes #422

Test plan

  • npx tsc --noEmit
  • npx jest src/app/features/user
  • npx prettier --check on the touched files

@zigzagdev zigzagdev self-assigned this Jul 26, 2026

@zigzagdev zigzagdev left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Ok

@zigzagdev
zigzagdev merged commit ae3da53 into main Jul 26, 2026
1 check passed
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.

User削除

1 participant