Skip to content

[WIP] Update templates to use inject instead of constructor - #5138

Closed
timdeschryver with Copilot wants to merge 1 commit into
mainfrom
copilot/update-templates-inject-method
Closed

[WIP] Update templates to use inject instead of constructor#5138
timdeschryver with Copilot wants to merge 1 commit into
mainfrom
copilot/update-templates-inject-method

Conversation

Copilot AI commented Apr 5, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>Update templates to use inject instead of constructor</issue_title>
<issue_description>### Which @ngrx/* package(s) are relevant/related to the feature request?

schematics

Information

The schematic templates make use of constructor, this needs to be update to use the inject method.

This can be replaced in:

  • .\modules\schematics\src\component-store\files_name@dasherize@if-flat__name@dasherize_.store.ts.template
  • .\modules\schematics\src\data\files_name@dasherize@if-flat__name@dasherize_.service.ts.template
  • .\modules\schematics\src\effect\files_name@dasherize@if-flat__name@dasherize_.effects.ts.template

Describe any alternatives/workarounds you're currently using

No response

I would be willing to submit a PR to fix this issue

  • Yes
  • No</issue_description>

Comments on the Issue (you are @copilot in this section)

@netlify

netlify Bot commented Apr 5, 2026

Copy link
Copy Markdown

Deploy Preview for ngrx-io ready!

Name Link
🔨 Latest commit a865d5f
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-io/deploys/69d2931c39f7650008abaa78
😎 Deploy Preview https://deploy-preview-5138--ngrx-io.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.

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.

Update templates to use inject instead of constructor

2 participants