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

Fix typo in state tree image #59

Merged
merged 1 commit into from
Jan 12, 2018
Merged

Conversation

jkhedani
Copy link
Contributor

Updated the image with the typo correction and double-checked spelling on all the other images. Luckily, you won't need to redo the awesome 3D version you made as the typo is well below the crop. Apologies - I didn't mean to take up too much of your time with this!

@coveralls
Copy link

Coverage Status

Coverage remained the same at 96.101% when pulling f91d0b7 on jkhedani:master into 2e47179 on mweststrate:master.

@mweststrate
Copy link
Collaborator

No worries, I should have spotted it as well (but I am notoriously bad at spotting spelling errors)

@mweststrate mweststrate merged commit cb4daee into immerjs:master Jan 12, 2018
@mweststrate
Copy link
Collaborator

If there is ever another revision, you could drop one of the leaf red dots; not all proxies have to have modifications. But that is so subtle, don't fill obliged to change that

@mweststrate
Copy link
Collaborator

Blog post is updated

@jkhedani
Copy link
Contributor Author

Ah, yup. No worries - I'll make the edit and submit another pull request.

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.

None yet

3 participants