Skip to content

Conversation

adamhollett
Copy link
Contributor

WHY are these changes introduced?

Removes the term "Isis" from Card examples.

I know that when it's not capitalized it just means an Egyptian goddess, but I don't think that's a distinction that users should have to make.

WHAT is this pull request doing?

I simply changed "Isis" to "Oasis" in these examples, to stay on-theme.

🎩 checklist

  • Updated the component's README.md with documentation changes

This is just a content change.

@ghost
Copy link

ghost commented Jan 28, 2020

👋 Thanks for opening your first pull request. A contributor should give feedback soon. If you haven’t already, please check out the contributing guidelines.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 28, 2020

🟢 No significant changes to src/**/*.tsx were detected.

Copy link
Contributor

@LauraAubin LauraAubin left a comment

Choose a reason for hiding this comment

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

I approved your Percy changes, and it looks like you just need to add a note to unreleased.md before shipping.

@adamhollett
Copy link
Contributor Author

Thanks Laura! I'll take care of that.

Co-Authored-By: Laura Aubin <laura.aubin@shopify.com>
@adamhollett adamhollett merged commit 53b0553 into master Jan 28, 2020
@adamhollett adamhollett deleted the isis-to-oasis branch January 28, 2020 17:57
@ghost
Copy link

ghost commented Jan 28, 2020

🎉 Thanks for your contribution to Polaris React!

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