Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify forkjoin error behavior #274

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tongfa
Copy link

@tongfa tongfa commented May 21, 2021

I read the learn-rxjs documentation on forkJoin, then read the main documentation at:
https://rxjs.dev/api/index/function/forkJoin

which states:
"""
If any given observable errors at some point, forkJoin will error as well and immediately unsubscribe from the other observables.
"""

I felt that the learn-rxjs documentation did not properly reflect that statement. After reading the learn-rxjs documentation I was under the impression all the inner observables would have completed successfully, including any side effects (i.e. server side state change) they may have caused. But that's not true.

This PR is my suggestion on how to improve the learn-rxjs documentation to properly reflect what the main documentation states.

I also fixed a couple example links that were in the same area.

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.

None yet

1 participant