Skip to content

align Angular User Management UI with standard entity templates#32359

Open
DhairyasheelS wants to merge 7 commits intojhipster:mainfrom
DhairyasheelS:angular-user-management-standard-templates
Open

align Angular User Management UI with standard entity templates#32359
DhairyasheelS wants to merge 7 commits intojhipster:mainfrom
DhairyasheelS:angular-user-management-standard-templates

Conversation

@DhairyasheelS
Copy link

@DhairyasheelS DhairyasheelS commented Feb 17, 2026

Use the same entity templates for User Management as for regular entities instead of custom admin/user-management templates. Add builtInUserManagement branching in routes, list, detail, update, delete, service, and routing resolve so login-based routing, setActive, and authorities keep working.

Closes #31891


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@github-actions github-actions bot added theme: maven theme: dependencies Pull requests that update a dependency file labels Feb 17, 2026
@DhairyasheelS DhairyasheelS force-pushed the angular-user-management-standard-templates branch from 861e91a to 3221be9 Compare February 18, 2026 21:04
@mshima
Copy link
Member

mshima commented Feb 22, 2026

@DhairyasheelS following #32387 and #32379 many customizations should be dropped.
We may need to adjusts to authorities handling before merging this PR.

@DhairyasheelS DhairyasheelS force-pushed the angular-user-management-standard-templates branch from 270e958 to 8f0700d Compare February 23, 2026 06:59
@DhairyasheelS
Copy link
Author

Thanks @mshima for the feedback.

I’ve reviewed #32387 and #32379 and understand that several customizations in this PR diverge from the standard entity template approach.

I will:

  • Drop the unnecessary custom logic and align strictly with the entity templates.
  • Refactor authorities handling to match the current JHipster security model and avoid any custom role branching.
  • Ensure routing, guards, and authority checks follow the generator conventions.

I pushed incremental updates . Please let me know if there’s any specific pattern from those PRs you’d like me to follow more closely.

DhairyasheelS added a commit to DhairyasheelS/DhairyasheelS-generator-jhipster that referenced this pull request Feb 24, 2026
…tity templates

- Add dedicated user-management template structure (entities/admin/user-management)
  with list, update, detail, delete, service, route and 20 template files
- Use UserManagementService and User type in list component (no dynamic generics)
- Customize user-management service: api/users, getUserIdentifier, UserManagementService
- Set entityPage to 'entities/admin/user-management' for user management routes
- Use userManagementEntityFiles section when writing user management UI
- Relax code-workspace generator type (settings/launch to any) for ESLint

Co-authored-by: Cursor <cursoragent@cursor.com>
DhairyasheelS and others added 7 commits March 5, 2026 00:45
- Use entity templates for admin/user-management instead of custom templates
- Add builtInUserManagement branching in routes, list, detail, update, delete, service, routing resolve
- Fix login regex in form service (escape backtick, raw EJS output)
- Add jhipster-keycloak-test to .gitignore

Closes jhipster#31891

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…opy chain, list filter

- List: trackIdentity returns item.login (string) when builtInUserManagement
- List: infinite-scroll deduplication uses login when builtInUserManagement
- List HTML: filter button queryParams use login; add hideListView to field filter
- Service: PartialUpdate type uses login when builtInUserManagement
- Service: single copy chain for create/update/partialUpdate when date + user mgmt
- Service: update() response conversion for builtInUserManagement; fix search() pipe
- List spec: trackIdentity test when builtInUserManagement; sort/defaultSort for id

Co-authored-by: Cursor <cursoragent@cursor.com>
- List HTML: remove hideListView from field filter (property not in model)
- List spec: use literal entity with login for trackIdentity test when builtInUserManagement

Co-authored-by: Cursor <cursoragent@cursor.com>
…ority import, trackIdentity via service, spec update

Co-authored-by: Cursor <cursoragent@cursor.com>
…tity templates

- Add dedicated user-management template structure (entities/admin/user-management)
  with list, update, detail, delete, service, route and 20 template files
- Use UserManagementService and User type in list component (no dynamic generics)
- Customize user-management service: api/users, getUserIdentifier, UserManagementService
- Set entityPage to 'entities/admin/user-management' for user management routes
- Use userManagementEntityFiles section when writing user management UI
- Relax code-workspace generator type (settings/launch to any) for ESLint

Co-authored-by: Cursor <cursoragent@cursor.com>
…r CI

- Use angularFiles.client for builtInUserManagement (same as other entities)
- Routes: always use UserRouteAccessService and canActivate; no defaultSort for user-management list
- List spec: use login for sort when builtInUserManagement (login,asc/desc)
- List: single trackPrimaryKey (trackLogin) for all entities; remove trackIdentity branch
- Unify trackPrimaryKey test in list spec for builtIn and regular entities

Made-with: Cursor
@DhairyasheelS DhairyasheelS force-pushed the angular-user-management-standard-templates branch from a28c165 to d7b3661 Compare March 4, 2026 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

theme: angular theme: dependencies Pull requests that update a dependency file theme: maven

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Angular: Align User Management UI generation with standard entity templates

3 participants