Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update createStore.ts #3749

Merged
merged 1 commit into from
Apr 13, 2020
Merged

Update createStore.ts #3749

merged 1 commit into from
Apr 13, 2020

Conversation

sktguha
Copy link
Contributor

@sktguha sktguha commented Apr 13, 2020

Replace broken link, "https://redux.js.org/api-reference/store#subscribelistener" link in the error message, by proper link "https://redux.js.org/api/store#subscribelistener"
Also I checked the current repo for any other such broken link but didn't find any


name: "\U0001F4DD Documentation Fix"
about: Fixing a problem in an existing docs page

Checklist

  • Is there an existing issue for this PR?
    • link issue here no such issue
  • Have the files been linted and formatted?
    yes

What docs page needs to be fixed?

  • Section: N/A
  • Page: N/A

What is the problem?

There is a broken link in the error message 

What changes does this PR make to fix the problem?

The PR changes the broken link in the error message to the correct one

Replace broken link, "https://redux.js.org/api-reference/store#subscribelistener" link in the error message, by proper link "https://redux.js.org/api/store#subscribelistener"
Also I checked the current repo for any other such broken link but didn't find any
@netlify
Copy link

netlify bot commented Apr 13, 2020

Deploy preview for redux-docs ready!

Built with commit c96116a

https://deploy-preview-3749--redux-docs.netlify.com

@markerikson markerikson merged commit 0040f45 into reduxjs:master Apr 13, 2020
@markerikson
Copy link
Contributor

Thanks!

@sktguha
Copy link
Contributor Author

sktguha commented Apr 13, 2020

welcome :)

@reduxjs reduxjs deleted a comment from sktguha Aug 25, 2020
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
Replace broken link, "https://redux.js.org/api-reference/store#subscribelistener" link in the error message, by proper link "https://redux.js.org/api/store#subscribelistener"
Also I checked the current repo for any other such broken link but didn't find any

Former-commit-id: df1767c
Former-commit-id: d5379c5
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
Replace broken link, "https://redux.js.org/api-reference/store#subscribelistener" link in the error message, by proper link "https://redux.js.org/api/store#subscribelistener"
Also I checked the current repo for any other such broken link but didn't find any

Former-commit-id: df1767c
Former-commit-id: d5379c5
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
Replace broken link, "https://redux.js.org/api-reference/store#subscribelistener" link in the error message, by proper link "https://redux.js.org/api/store#subscribelistener"
Also I checked the current repo for any other such broken link but didn't find any

Former-commit-id: df1767c
Former-commit-id: d5379c5
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.

2 participants