Skip to content

Add text to AppNameWarning about setting AppName #140

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

Conversation

Naturalclar
Copy link
Member

Summary

Finishing up #113 by adding information about being able to change the AppName from the settings.

Test Plan

What are the steps to reproduce?

Checklist

  • I tested this thoroughly

@@ -15,6 +15,8 @@ export const AppNameWarning = () => (
<Markdown>
Keep in mind that `RnDiffApp` and `rndiffapp` are placeholders. When
upgrading, you should replace them with your actual project's name.
You can also make these replacements by setting your app name from the
Copy link
Member

Choose a reason for hiding this comment

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

How about:

You can also provide your app name by clicking the settings icon on the top right.?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, nice and concise :)

Copy link
Member

@lucasbento lucasbento left a comment

Choose a reason for hiding this comment

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

Nice, thank you for another great contribution!

@lucasbento lucasbento merged commit f155d8e into react-native-community:master Jan 9, 2020
@Naturalclar Naturalclar deleted the chore/AddTextToAppWarning branch January 9, 2020 17:56
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