Skip to content

angular: migrate entity list to httpResource based service#32373

Merged
DanielFran merged 2 commits intojhipster:mainfrom
mshima:httpResource
Feb 21, 2026
Merged

angular: migrate entity list to httpResource based service#32373
DanielFran merged 2 commits intojhipster:mainfrom
mshima:httpResource

Conversation

@mshima
Copy link
Member

@mshima mshima commented Feb 19, 2026


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.

Copy link
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

This pull request migrates Angular entity list components to use Angular's new httpResource API for a more reactive and declarative approach to HTTP request management. The changes modernize the entity list loading mechanism by replacing imperative HttpClient calls with signal-based httpResource, introduce a new utility function for controlled pluralization, and update documentation to use more semantically appropriate entity name variants.

Changes:

  • Introduced a new pluralize utility function with a force parameter to control pluralization behavior, allowing distinction between display names (forced plurals) and URL/file names (natural plurals)
  • Migrated Angular entity list services to use httpResource with signal-based parameter management, replacing manual HttpClient subscription handling
  • Refactored service architecture to use inheritance pattern with a base plural service managing list loading and a child singular service adding CRUD operations
  • Updated Java template documentation to use entityNamePluralHumanized instead of entityInstancePlural for better semantic clarity

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
lib/utils/string-utils.ts Added pluralize utility function with force parameter for controlled pluralization behavior
generators/spring-boot/templates/src/main/java/package/entityPackage/web/rest/_entityClass_Resource.java.ejs Updated JavaDoc comments to use entityNamePluralHumanized for better documentation clarity
generators/java/generators/bootstrap/generator.ts Updated to use new pluralize function with force parameter for relationship name pluralization
generators/client/entity.ts Updated to use new pluralize function for Angular entity name pluralization
generators/base-application/entity.ts Introduced force parameter in pluralization calls and updated entityNamePluralHumanized computation
generators/angular/templates/src/main/webapp/app/entities/entityFolder/service/entityFile.service.ts.ejs Major refactoring to use httpResource API with signal-based parameters, introduced service inheritance pattern
generators/angular/templates/src/main/webapp/app/entities/entityFolder/list/entityFile.ts.ejs Updated component to work with new httpResource-based service, removed manual HTTP subscription handling
generators/angular/templates/src/main/webapp/app/entities/entityFolder/list/entityFile.spec.ts.ejs Updated tests to work with httpResource using HttpTestingController and vitest fake timers
generators/angular/templates/src/main/webapp/app/entities/entityFolder/list/entityFile.html.ejs Fixed currentSearch signal invocation to use function call syntax

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mshima mshima assigned OmarHawk and unassigned OmarHawk Feb 19, 2026
@mshima mshima requested a review from OmarHawk February 19, 2026 19:37
@mshima mshima marked this pull request as ready for review February 19, 2026 19:47
@DanielFran DanielFran merged commit bba9b84 into jhipster:main Feb 21, 2026
200 of 215 checks passed
@mshima mshima deleted the httpResource branch February 21, 2026 16:39
@mraible mraible added this to the 9.0.0 milestone Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants