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

New UI, Old docs #1338

Merged
merged 12 commits into from
Apr 2, 2020
Merged

New UI, Old docs #1338

merged 12 commits into from
Apr 2, 2020

Conversation

jordanranz
Copy link
Contributor

@jordanranz jordanranz commented Mar 26, 2020

Description of changes:

  • Amplify UI Components documentation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jordanranz jordanranz marked this pull request as ready for review April 2, 2020 15:53
_config.yml Outdated Show resolved Hide resolved
excerpt: Version of the docs
cta: Read more
class: aws-icon-ps-60-circular-circuit
url: "/sdk/ios/start"
Copy link
Contributor

Choose a reason for hiding this comment

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

Seeing that most of these changes look like prettier doing something, do we want to do the same config change we did for JS last night?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is no prettier in docs v1 so I don't believe so. Not sure why my VS code formatted everything..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this is valid formatting though

Copy link
Contributor

Choose a reason for hiding this comment

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

Agreed it was more of a callout if it was something with Prettier, which it isn't, and also just seeing what it was.

Copy link
Contributor

Choose a reason for hiding this comment

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

This was non-blocking btw

Copy link
Contributor

@sammartinez sammartinez left a comment

Choose a reason for hiding this comment

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

LGTM 🌮

@jordanranz jordanranz merged commit e302880 into master Apr 2, 2020
@renebrandel renebrandel deleted the ui-components branch April 2, 2020 23:20
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.

3 participants