Skip to content

Conversation

@ykethan
Copy link
Member

@ykethan ykethan commented Apr 24, 2023

Description of changes:

minor compilation of wording and formatting fixes

Related GitHub issue #, if available:

Instructions

If this PR should not be merged upon approval for any reason, please submit as a DRAFT

Which product(s) are affected by this PR (if applicable)?

  • amplify-cli
  • amplify-ui
  • amplify-studio
  • amplify-hosting
  • amplify-libraries

Which platform(s) are affected by this PR (if applicable)?

  • JS
  • iOS
  • Android
  • Flutter
  • React Native

Please add the product(s)/platform(s) affected to the PR title

Checks

  • Does this PR conform to the styleguide?

  • Does this PR include filetypes other than markdown or images? Please add or update unit tests accordingly.

  • Are any files being deleted with this PR? If so, have the needed redirects been created?

  • Are all links in MDX files using the MDX link syntax rather than HTML link syntax?

    ref: MDX: [link](https://link.com)
    HTML: <a href="https://link.com">link</a>

When this PR is ready to merge, please check the box below

  • Ready to merge

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ykethan ykethan requested a review from a team as a code owner April 24, 2023 22:07
@ykethan ykethan added the amplify/studio Issues and PRs related to amplify studio documentation label Apr 24, 2023
@ykethan ykethan requested review from a team and wrpeck April 24, 2023 22:07
@aws-amplify-us-west-2
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-5424.d39agkzf0cwtdz.amplifyapp.com

@katieklein katieklein added product-owner-review-needed Current step in the approval process requires a PM review technical-review-needed labels Apr 25, 2023
@ykethan ykethan requested a review from wrpeck April 26, 2023 12:08
@ykethan ykethan requested a review from cwoolum April 29, 2023 14:00
Copy link

@letsbelopez letsbelopez left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@wrpeck wrpeck left a comment

Choose a reason for hiding this comment

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

Nice work Sai

@ykethan ykethan removed product-owner-review-needed Current step in the approval process requires a PM review technical-review-needed labels May 5, 2023
@katieklein katieklein merged commit 25a348e into aws-amplify:main May 9, 2023
@ykethan ykethan deleted the studio-dx-improvements branch May 10, 2023 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

amplify/studio Issues and PRs related to amplify studio documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants