Skip to content

Conversation

samwightt
Copy link
Contributor

@samwightt samwightt commented Jul 14, 2020


name: "Fix type error in Redux Essentials"
about: Update documentation content as per #3812

Checklist

What docs page needs to be fixed?

  • Showing Single Posts:
  • Redux Essentials, Part 4: Using Redux Data:

What is the problem?

This fixes a type error in the documentation by updating the IDs in the initial store data to be strings, not numbers. This removes the need for .toString() methods.

What changes does this PR make to fix the problem?

Removes the .toString() method from Part 4 of the tutorial and updated the initial store data in Part 3 of the tutorial to use strings for IDs.

@netlify
Copy link

netlify bot commented Jul 14, 2020

Deploy preview for redux-docs ready!

Built with commit 434b20b

https://deploy-preview-3814--redux-docs.netlify.app

@timdorr
Copy link
Member

timdorr commented Jul 14, 2020

Thanks!

@timdorr timdorr merged commit b08cf38 into reduxjs:master Jul 14, 2020
alokreddy pushed a commit to alokreddy/redux that referenced this pull request Sep 30, 2020
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
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