Skip to content

ActivityIndicator: add snackplayer/functional component with hooks#1590

Merged
nearestnabors merged 5 commits intofacebook:masterfrom
ronnycoding:patch-1
Feb 15, 2020
Merged

ActivityIndicator: add snackplayer/functional component with hooks#1590
nearestnabors merged 5 commits intofacebook:masterfrom
ronnycoding:patch-1

Conversation

@ronnycoding
Copy link
Copy Markdown
Contributor

Open Issue: #1579

  • Added snackplayer functional example for ActivityIndicator component

@ronnycoding ronnycoding changed the title added snackplayer/functional component with hooks added snackplayer/functional component with hooks (ActivityIndicator) Feb 5, 2020
@ronnycoding ronnycoding mentioned this pull request Feb 5, 2020
@react-native-bot
Copy link
Copy Markdown

react-native-bot commented Feb 5, 2020

Oops, something went wrong with this deploy preview.

@charpeni can you look at this?

@ronnycoding ronnycoding changed the title added snackplayer/functional component with hooks (ActivityIndicator) ActivityIndicator: added snackplayer/functional component with hooks Feb 5, 2020
@ronnycoding ronnycoding requested review from hramos and kelset February 6, 2020 21:21
Copy link
Copy Markdown
Contributor

@nearestnabors nearestnabors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ach, you've don't this in the versioned docs! We need the file to be edited here:
https://github.com/facebook/react-native-website/blob/master/docs/activityindicator.md

"toastandroid": {
"title": "ToastAndroid"
},
"toolbarandroid": {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ronnyiscoding Same problem here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR updated here 12166a4 thanks!

Copy link
Copy Markdown
Collaborator

@Simek Simek Feb 7, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the changes.

As @rachelnabors said (which apply for the both PRs) you should update:

  • master docs (docs/activityindicator.md)

not:

  • versioned docs (website/versioned_docs/version-0.61/activityindicator.md)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, Thank you, PR updated at 51913e4

@ronnycoding ronnycoding changed the title ActivityIndicator: added snackplayer/functional component with hooks ActivityIndicator: add snackplayer/functional component with hooks Feb 7, 2020
@ronnycoding
Copy link
Copy Markdown
Contributor Author

Ach, you've don't this in the versioned docs! We need the file to be edited here:
https://github.com/facebook/react-native-website/blob/master/docs/activityindicator.md

Thanks for the review, PR is updated please let me know if this is the right way to contribute 👍

Copy link
Copy Markdown
Contributor

@nearestnabors nearestnabors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done! Thanks for adding these examples!

Prettifying the code
@react-native-bot
Copy link
Copy Markdown

Deploy preview for react-native ready!

Built with commit caedf1b

https://deploy-preview-1590--react-native.netlify.com

Changes to docs/ are reflected in the next "master" version.

Thank you for your contributions.

How to ContributeDocumentation Sources

@nearestnabors nearestnabors merged commit 9fc1183 into facebook:master Feb 15, 2020
kaioduarte pushed a commit to kaioduarte/react-native-website that referenced this pull request Feb 24, 2020
…ebook#1590)

* added snackplayer/functional component with hooks

* update docs

* restore changes

* PR updates

* Update activityindicator.md

Prettifying the code

Co-authored-by: Rachel Nabors <rachelnabors@users.noreply.github.com>
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