Skip to content

fix: add admin user CRUD and resolve management console issues - #45

Merged
DavidHLP merged 1 commit into
mainfrom
feat/fix-management-user-issues
May 17, 2026
Merged

fix: add admin user CRUD and resolve management console issues#45
DavidHLP merged 1 commit into
mainfrom
feat/fix-management-user-issues

Conversation

@DavidHLP

Copy link
Copy Markdown
Owner

Summary

  • Add admin user CRUD operations (create, update, delete, reset password)
  • Fix management console user dialog issues
  • Update i18n translations for user module (zh-CN, en-US)
  • Add backend DTOs and service layer for admin user management

Changes

  • Backend: AdminUserController, AdminUserService, AdminCreateUserDTO, AdminUpdateUserDTO
  • Frontend: UserCreateDialog.vue, UserEditDialog.vue, UserResetPasswordDialog.vue
  • i18n: Added missing user module translation keys

Test plan

  • Verify admin user creation flow
  • Verify admin user update flow
  • Verify password reset functionality
  • Check i18n label rendering in both languages

🤖 Generated with Claude Code

- Add createUser, updateUser, deleteUser endpoints to AdminUserController
- Add username/email uniqueness checks in createUser and updateUser
- Add @pattern validation for role field in both create and update DTOs
- Add @notblank to email in AdminCreateUserDTO
- Fix UUID import in AdminUserServiceImpl
- Add missing i18n keys for user edit and reset password dialogs
- Fix DialogDescription a11y warnings in user dialogs
- Add client-side password length validation before reset

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@DavidHLP
DavidHLP merged commit c3ee8ce into main May 17, 2026
6 of 10 checks passed
@DavidHLP
DavidHLP deleted the feat/fix-management-user-issues branch May 17, 2026 14:22
DavidHLP added a commit that referenced this pull request Jul 3, 2026
fix: add admin user CRUD and resolve management console issues
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.

1 participant