search

baconjs / baconjs.github.io

bacon.js website
8 stars 19 forks source link

Wrong link in documentation: Bacon.mergeAll() -> observable.merge() #40

Closed alinnert closed 6 years ago

alinnert commented 6 years ago

https://baconjs.github.io/api2.html#bacon-mergeall

This part of the documentation says:

merges given array of EventStreams or Properties. Returns an EventStream. See merge

The last word "merge" is a link containing the anchor #observable-merge. This anchor doesn't exist though. But there's an anchor called #stream-merge.

I don't know, is this the intended target of the link behind "merge"?

This issue is present in both v1 and v2 api documentation.

raimohanska commented 6 years ago

Thanks. Fixed!