search

ReactiveX / rxjs

A reactive programming library for JavaScript
https://rxjs.dev
Apache License 2.0
30.7k stars 3k forks source link

Alias `switchMap` as `flatMapLatest`? #760

Closed benlesh closed 8 years ago

benlesh commented 8 years ago

I think a lot of people that are familiar with Rx are going to try to use flatMapLatest and be confused. Should we alias it? If so, should we add a deprecation warning?

staltz commented 8 years ago

I vote for moving to the new name, and having zero (or as close to zero as possible) aliases in the whole operator repertoire. The confusion aliases add last longer than a short migration confusion.

Frikki commented 8 years ago

I also support no aliases.

kwonoj commented 8 years ago

Generally in favor of no aliases, but docs or somewhere else might need to state it's deprecated such as RxJS4 does (https://github.com/Reactive-Extensions/RxJS/blob/master/doc/api/core/operators/fromarray.md)

staltz commented 8 years ago

Yes, definitely we should mention.

benlesh commented 8 years ago

That all sounds fine. I'm going to leave this up for another week (after this comment), then I'm going to close it.

benlesh commented 8 years ago

closing for inactivity.