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

Migrate vuex to pinia #3575

Merged
merged 11 commits into from
Jun 23, 2024
Merged

Migrate vuex to pinia #3575

merged 11 commits into from
Jun 23, 2024

Conversation

dartcafe
Copy link
Collaborator

Step one of Vue3 migration

  • replace vuex with pinia
  • Adding some temporary tweaks
  • removed acl in favor of a session store
  • router is temporarily injected into the sessionstore by watchers
  • used mapStores() for lazy store usage

Possibly some bugs are left.

dartcafe added 9 commits June 2, 2024 22:04
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe dartcafe added this to the 8.0 milestone Jun 23, 2024
@dartcafe dartcafe changed the title Temp/next/pinia/replace-vuex Migrate vuex to pinia Jun 23, 2024
dartcafe added 2 commits June 23, 2024 22:26
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe dartcafe merged commit e3b30d5 into next Jun 23, 2024
17 checks passed
@dartcafe dartcafe deleted the temp/next/pinia/replace-vuex branch November 3, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant