Skip to content

refactor(pinia): move remaining vuex state to pinia setting store - #8259

Open
grnd-alt wants to merge 1 commit into
mainfrom
refactor/pinia-migration-mainStore-the-second
Open

refactor(pinia): move remaining vuex state to pinia setting store#8259
grnd-alt wants to merge 1 commit into
mainfrom
refactor/pinia-migration-mainStore-the-second

Conversation

@grnd-alt

@grnd-alt grnd-alt commented Aug 11, 2026

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200129 was 198420 (+0.86%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@grnd-alt
grnd-alt force-pushed the refactor/pinia-migration-mainStore-the-second branch 2 times, most recently from 8b21a99 to 3acfa0d Compare August 11, 2026 13:53
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200149 was 198420 (+0.87%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200147 was 198420 (+0.87%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@grnd-alt
grnd-alt force-pushed the refactor/pinia-migration-mainStore-the-second branch from 3acfa0d to 468a56f Compare August 12, 2026 07:31
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200146 was 198420 (+0.86%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@grnd-alt
grnd-alt force-pushed the refactor/pinia-migration-mainStore-the-second branch from 468a56f to db95798 Compare August 12, 2026 09:44
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200112 was 198420 (+0.85%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@grnd-alt
refactor(pinia): move remaining vuex state to pinia setting store
2199a3a 
Signed-off-by: grnd-alt <git@belakkaf.net>
@grnd-alt
grnd-alt force-pushed the refactor/pinia-migration-mainStore-the-second branch from db95798 to 2199a3a Compare August 12, 2026 10:47
@grnd-alt
grnd-alt marked this pull request as ready for review August 12, 2026 10:49
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200148 was 198420 (+0.87%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@grnd-alt
grnd-alt requested a review from a team August 12, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luka-nextcloud luka-nextcloud Awaiting requested review from luka-nextcloud luka-nextcloud is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@grnd-alt