Skip to content

refactor: remove Vuex dependency by migrating user_status to Pinia - #66151

Merged
skjnldsv merged 2 commits into
masterfrom
refactor/user-status-pinia
Nov 2, 2026
Merged

skjnldsv merged 2 commits into
masterfrom
refactor/user-status-pinia

Conversation

@susnux

@susnux susnux commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Migrate last usage of Vuex to Pinia.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@susnux susnux added 3. to review Waiting for reviews feature: status ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Nov 2, 2026
@susnux susnux added this to the Nextcloud 36 milestone Nov 2, 2026
@susnux
susnux marked this pull request as ready for review November 2, 2026 16:45
@susnux
susnux requested review from a team as code owners November 2, 2026 16:45
@susnux
susnux requested review from kristian-zendato, pringelmann and skjnldsv and removed request for a team November 2, 2026 16:46
@susnux susnux changed the title Refactor/user status pinia refactor: remove Vuex dependency by migrating user_status to Pinia Nov 2, 2026
@susnux
susnux requested a review from szaimen November 2, 2026 20:07
Replace the vuex store with pinia setup stores, written in TypeScript.
The user status and backup status stores share their state handling
through a common composable. Both apps mounted by the menu entry point
(user menu and dashboard status) keep sharing one pinia instance.

Saving a predefined status message now awaits the request, so errors are
shown and the modal stays open, like for custom messages.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
No app built by the Vue 3 frontend uses vuex anymore. The legacy
frontend keeps its own vuex 3 dependency.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnux force-pushed the refactor/user-status-pinia branch from 74073e3 to 492e2f9 Compare November 2, 2026 20:14

@szaimen szaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🐘

@skjnldsv
skjnldsv merged commit dec3834 into master Nov 2, 2026
86 checks passed
@skjnldsv
skjnldsv deleted the refactor/user-status-pinia branch November 2, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted feature: status ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants