Skip to content

docs: fix rxResource example of validateAsync#68235

Open
michael-small wants to merge 1 commit intoangular:mainfrom
michael-small:patch-5
Open

docs: fix rxResource example of validateAsync#68235
michael-small wants to merge 1 commit intoangular:mainfrom
michael-small:patch-5

Conversation

@michael-small
Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Invalid destructured stream > request for the rxResource, and the respective validateAsync has an invalid fallback onto || undefined when the schema's value() will be a defined string already.

Issue Number: N/A

What is the new behavior?

stream > params and no || undefined fallback for the validateAsync.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@pullapprove pullapprove bot requested a review from crisbeto April 16, 2026 02:50
@angular-robot angular-robot bot added the area: docs Related to the documentation label Apr 16, 2026
@ngbot ngbot bot added this to the Backlog milestone Apr 16, 2026
Copy link
Copy Markdown
Member

@JeanMeche JeanMeche left a comment

Choose a reason for hiding this comment

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

Thanks for this

@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Apr 16, 2026
@JeanMeche JeanMeche removed the request for review from crisbeto April 16, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants