Skip to content

Conversation

ErikCH
Copy link
Contributor

@ErikCH ErikCH commented Oct 17, 2022

Issue #, if available:

Description of changes: This updates the getting started guide for Vue.js. Fixes some issues with the guide.

  • Getting started guide was using the older Vue CLI which is now in maintenance mode. Updated guide to use Vue's create-vue which is the preferred way of creating Vue apps.
  • Fixed bug in data store guide that called subscription that wasn't there.

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

@ErikCH ErikCH requested a review from a team as a code owner October 17, 2022 18:09
name: 'App',
async created() {
this.getTodos();
this.subscribe();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was the only real change in this file. This was calling subscribe before it was added.

@@ -1,17 +1,25 @@
Use the Vue CLI to bootstrap a new Vue 3 app (selecting the defaults will work for this project):
Use the Vue Vite powered `create-app` to bootstrap a new Vue 3 app (selecting the defaults will work for this project):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved guides to use the new create-app utility for Vue 3.

## Vue Vite Config

Now that our Vue app is set up and Amplify is initialized, we're ready to add an API in the next step.
When working with a Vue `create-app` that uses [Vite](https://vitejs.dev) you must make a few additional modifications. Please follow the steps below.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Vite requires a little more configuration

},
]
}
})
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice!!!! This lines up with what we are recommending to customers as well - it will be nice to have a section in our docs we can refer customers to. I am probably going to take this part and add it to the troubleshooting side of the JS library guide. kevinold/repro-vite-amplify@4d6b422

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK great!

@ErikCH ErikCH merged commit 62d3395 into aws-amplify:main Oct 19, 2022
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