-
Notifications
You must be signed in to change notification settings - Fork 3k
Description
Since RxJS v7 has been released, many operators that accept "notifiers" have been adjusted to listen to only "next" notifications. This has become mandatory as per RxJS Core Semantics guide:
Operators that accept "notifiers" (as described above), MUST ONLY recognized next values from the notifier as "notifications". Emitted completions may not be used a source of notification.
However, many marble diagram images have to be updated as they don't reliably picture this behavior. For example, audit
operator displays this image:
A user "reading" this image might be deceived to think that an Observable that has been passed to audit
operator needs to complete before destination will receive next "next" notification. This is not true now.
Marble diagram images of these operators need to be updated:
- audit (fixed in PR: docs(audit): fix marble diagram #6636)
- debounce (fixed in PR: docs(debounce): fix marble diagram #6639)
- throttle (fixed in PR: docs(throttle): fix marble diagram #6638)
- bufferWhen (fixed in PR: docs(bufferWhen): fix marble diagram #6767)
- windowWhen (PR: fix(windowWhen): don't signal on complete #7052)