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

Merge head #1

Merged
Merged

Conversation

trchopan
Copy link
Owner

@trchopan trchopan commented Jul 2, 2022

No description provided.

posva and others added 22 commits May 19, 2022 14:31
Corrected the getter name for full name in example of usage vuex inside the componets
I think this comma is unintended
Use the correct subdomain to link to the Vue 2 documentation. Since the switch to Vue 3 as the default Vue version, to access the Vue 2 documentation the `https://v2.vuejs.org` subdomain must be used.
Co-authored-by: Eduardo San Martin Morote <posva@users.noreply.github.com>
Correct typo on comment
Co-authored-by: Eduardo San Martin Morote <posva@users.noreply.github.com>
On https://pinia.vuejs.org/core-concepts/ it is mentioned that "Naming the returned function use... is a convention across composables to make its usage idiomatic.". In this example though, the returned function is called `todos` as opposed to `useTodos`.
Co-authored-by: Eduardo San Martin Morote <posva13@gmail.com>
Co-authored-by: Eduardo San Martin Morote <posva@users.noreply.github.com>
Co-authored-by: liuyao <liuyao@liuyaodeiMac.local>
@trchopan trchopan merged commit 7726981 into trchopan:usecase-for-function-style-in-define-store Jul 2, 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.