Skip to content

feat(signals): add delegatedSignal#5151

Merged
timdeschryver merged 2 commits into
mainfrom
feat/signals/delegated-signal
May 4, 2026
Merged

feat(signals): add delegatedSignal#5151
timdeschryver merged 2 commits into
mainfrom
feat/signals/delegated-signal

Conversation

@markostanimirovic
Copy link
Copy Markdown
Member

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?

Closes #5121

Does this PR introduce a breaking change?

[ ] Yes
[x] No

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 18, 2026

Deploy Preview for ngrx-io ready!

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

Copy link
Copy Markdown
Member

@timdeschryver timdeschryver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
To also include this in the API docs you should run collect-docs, I'm thinking about creating an issue to look into automating this because it's often forgotten.

Copy link
Copy Markdown
Contributor

@rainerhahnekamp rainerhahnekamp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left one comment for the docs. feel free to commit or skip. Otherwise all good for me!

Comment thread projects/www/src/app/pages/guide/signals/delegated-signal.md Outdated
Co-authored-by: Rainer Hahnekamp <rainer.hahnekamp@gmail.com>
@timdeschryver timdeschryver merged commit a469cbf into main May 4, 2026
10 checks passed
@timdeschryver timdeschryver deleted the feat/signals/delegated-signal branch May 4, 2026 18:50
@michael-small
Copy link
Copy Markdown
Contributor

Sorry to dig this up, but I imagine it may be overkill to make this a whole issue:

Was it intended for the doc page to be live already? https://ngrx.io/guide/signals/delegated-signal

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.

RFC(@ngrx/signals): Add delegatedSignal API

4 participants