Skip to content

Conversation

@michaelg100
Copy link
Contributor

@michaelg100 michaelg100 commented Jul 5, 2024

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Docs
  • Other... Please describe:

What is the current behavior?

Currently the observable streams url point to: http://reactivex.io/rxjs/class/es6/Observable.js~Observable.html. However if you're on that site and click home it will bring you to their newer doc site.

Also a few things are currently misspelled.

Issue Number: N/A

What is the new behavior?

Updating the link to point to the latest observable docs here: https://rxjs-dev.firebaseapp.com/guide/observable.

Also, fixing some misspellings.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

N/A

@michaelg100 michaelg100 changed the title Update RxJS Observable URL to newer version Update RxJS Observable URL to newer version + Fix some spelling mistakes Jul 5, 2024
@kamilmysliwiec kamilmysliwiec merged commit 43b4ea5 into nestjs:master Jul 8, 2024
@kamilmysliwiec
Copy link
Member

lgtm

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.

3 participants