Update the API entries for mixins and extends to reference setup #1129

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.

Sign up for GitHub

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

Jump to bottom

Closed

skirtles-code wants to merge 1 commit into vuejs:master from skirtles-code:setup-extends-mixins

Contributor

skirtles-code commented Jul 7, 2021

I've updated the API reference entries for mixins and extends:

Mention the composition API in the entry for mixins.
Reword the entry for extends to better reflect Vue 3. Using a function/constructor is no longer a thing and Vue.extend doesn't exist anymore.
Add an explanation of how to combine extends and setup. This is based on a couple of conversations on Discord where 'don't use extends' didn't seem like a legitimate suggestion.

Marking as a draft for now as I want to check that last part isn't too controversial.


          docs: update the API entries for mixins and extends to reference setup

301958d

skirtles-code mentioned this pull request

Update the API entries for mixins and extends #1146

Merged

Contributor Author

skirtles-code commented Jul 20, 2021

Closing this.

The less controversial parts are now in #1146. I'll rethink the part about combining extends and setup

skirtles-code closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet