Skip to content

Fix Platform.OS reference#1812

Closed
pfertyk wants to merge 1 commit into
react:masterfrom
pfertyk:patch-1
Closed

Fix Platform.OS reference#1812
pfertyk wants to merge 1 commit into
react:masterfrom
pfertyk:patch-1

Conversation

@pfertyk

@pfertyk pfertyk commented Apr 2, 2020

Copy link
Copy Markdown
Contributor

The reference to Platform.Os is incorrect, it should be Platform.OS. It is a small one, but it costed me some time (I thought that KeyboardAvoidingView is not working, but it just used a wrong setting for iOS, because of this typo ;) ).

@vonovak

vonovak commented Apr 2, 2020

Copy link
Copy Markdown
Contributor

maintainers: please note this fix (and more fixes) are included in #1803

@pfertyk

pfertyk commented Apr 2, 2020

Copy link
Copy Markdown
Contributor Author

I didn't know there is another PR for the same fix, thanks for the link ;)

@Simek

Simek commented Apr 4, 2020

Copy link
Copy Markdown
Collaborator

@pfertyk #1803 was merged, can you close this PR?

@pfertyk

pfertyk commented Apr 4, 2020

Copy link
Copy Markdown
Contributor Author

@Simek great news, thanks for letting me know ;) Do you know when will the documentation be updated? The typo is still there: https://reactnative.dev/docs/keyboardavoidingview

@Simek

Simek commented Apr 4, 2020

Copy link
Copy Markdown
Collaborator

@pfertyk This issue needs a PR for 0.62 versioned docs. Changes from the master docs will be included in next generated version (0.63). You can currently see those changes in next docs but they are not automatically ported backwards.

@nearestnabors

Copy link
Copy Markdown
Contributor

Thanks for submitting! I know we couldn't accept this one, but it would be great if you could edit 0.62's docs to include the same change, @pfertyk!

@pfertyk

pfertyk commented Apr 6, 2020

Copy link
Copy Markdown
Contributor Author

Thank for the feedback @rachelnabors , I will send a new PR soon ;)

@pfertyk

pfertyk commented Apr 7, 2020

Copy link
Copy Markdown
Contributor Author

@rachelnabors PR submitted: #1826, please correct me if that was the wrong file to edit for 0.62 ;)

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.

4 participants