Skip to content

Conversation

@timdeschryver
Copy link
Member

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[x] Bugfix
[ ] 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?

Closes #5073

The IdSelector always returned string | number.

What is the new behavior?

The return type if inferred by the adapter:

  • uses the id property of the state
  • uses the return type of the selectId method, if provided in the config options

Does this PR introduce a breaking change?

I'm also ok with flagging this as a breaking change.

[ ] Yes
[x] No

Other information

@netlify
Copy link

netlify bot commented Feb 7, 2026

Deploy Preview for ngrx-io ready!

Name Link
🔨 Latest commit 0c3ea13
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-io/deploys/69874165a01c5b0008848515
😎 Deploy Preview https://deploy-preview-5085--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.

[Entity] Expose IdSelectorNum and IdSelectorStr types for better type safety

1 participant