Skip to content

feat(signals): add ability to provide SignalStore at the platform level#4964

Merged
timdeschryver merged 3 commits intongrx:mainfrom
MillerSvt:add-provided-in-platform
Oct 13, 2025
Merged

feat(signals): add ability to provide SignalStore at the platform level#4964
timdeschryver merged 3 commits intongrx:mainfrom
MillerSvt:add-provided-in-platform

Conversation

@MillerSvt
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Cannot pass any providedIn except root.

Closes #4963

What is the new behavior?

Added support to any not deprecated providedIn types.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

I was thinking of just adding the platform to the union. But it seems that if here we just pass the value into the Injectable decorator, then we should rely on the typing of this decorator. The only thing seems to be that we should exclude deprecated types from union, so as not to add support for what will be removed soon.

@netlify
Copy link

netlify bot commented Sep 24, 2025

Deploy Preview for ngrx-site-v19 ready!

Name Link
🔨 Latest commit 173e3b2
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-site-v19/deploys/68ebbe9a46190f0008bbc835
😎 Deploy Preview https://deploy-preview-4964--ngrx-site-v19.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.

@netlify
Copy link

netlify bot commented Sep 24, 2025

Deploy Preview for ngrx-io canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 173e3b2
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-io/deploys/68ebbe9ad728530008b8fe05

@MillerSvt MillerSvt force-pushed the add-provided-in-platform branch from d4ee15f to 40110c4 Compare October 4, 2025 15:44
@markostanimirovic markostanimirovic changed the title feat(signals): add support all providedIn types from Injectable (#4963) feat(signals): add ability to provide SignalStore at the platform level Oct 12, 2025
@timdeschryver
Copy link
Member

Thanks @MillerSvt

@timdeschryver timdeschryver merged commit 835014b into ngrx:main Oct 13, 2025
14 checks 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.

Add support providedIn: 'platform'

3 participants