Skip to content
This repository was archived by the owner on Apr 20, 2018. It is now read-only.

Commit 089dca5

Browse files
Merge pull request #636 from jmalonzo/jmalonzo-designguide-flatmapselectmany
Update usage of SelectMany to include flatMap
2 parents a47af00 + 3e6017c commit 089dca5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

doc/designguidelines/readme.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ In traditionally written programs, this debouncing would introduce separate call
9494

9595
Once the user input has been filtered down it is time to perform the dictionary lookup. As this is usually an expensive operation (e.g. a request to a server on the other side of the world), this operation is itself asynchronous as well.
9696

97-
The `flatMap`/`selectMany` operator allows for easy combining of multiple asynchronous operations. It doesn’t only combine success values; it also tracks any exceptions that happen in each individual operation.
97+
The `flatMap` or `selectMany` operator allows for easy combining of multiple asynchronous operations. It doesn’t only combine success values; it also tracks any exceptions that happen in each individual operation.
9898

9999
In traditionally written programs, this would introduce separate callbacks and a place for exceptions occurring.
100100

@@ -431,7 +431,7 @@ Rx.Observable.prototype.flatMap = function (selector) {
431431
};
432432
```
433433

434-
In this sample, the SelectMany operator uses two existing operators: `map` and `mergeAll`. The `map` operator already deals with any issues around the selector function throwing an exception. The `mergeAll` operator already deals with concurrency issues of multiple observable sequences firing at the same time.
434+
In this sample, the `flatMap` or `selectMany` operator uses two existing operators: `map` and `mergeAll`. The `map` operator already deals with any issues around the selector function throwing an exception. The `mergeAll` operator already deals with concurrency issues of multiple observable sequences firing at the same time.
435435

436436
#### When to ignore this guideline ####
437437

0 commit comments

Comments
 (0)