Page MenuHomePhabricator

Vuex store cleanup: Organize mixins and move auxiliary functions away from store modules to its relevant helper file
Closed, ResolvedPublic

Description

Description

The goal of this task is to keep clean and clear all the Vuex store module, so that they only have the declaration of state, getters, mutations and actions.
For this, the suggestion is to take all the auxiliary functions that we stack on top of these files and take them somewhere else, a utils or helpers file, or even one of the mixins depending on whether they are used from inside the components or not.


Completion checklist

Event Timeline

Jdforrester-WMF changed the task status from Open to In Progress.Feb 5 2024, 6:04 PM
gengh renamed this task from Vuex store cleanup: Move auxiliary functions from store modules to a utils.js or helpers.js files to Vuex store cleanup: Organize mixins and move auxiliary functions from store modules to a utils.js or helpers.js files.Feb 6 2024, 10:16 AM
gengh renamed this task from Vuex store cleanup: Organize mixins and move auxiliary functions from store modules to a utils.js or helpers.js files to Vuex store cleanup: Organize mixins and move auxiliary functions away from store modules to its relevant helper file.

Change 997923 had a related patch set uploaded (by Genoveva Galarza; author: Genoveva Galarza):

[mediawiki/extensions/WikiLambda@master] Cleanup: Rename zobjectTreeUtils mixin and methods

https://gerrit.wikimedia.org/r/997923

Change 997924 had a related patch set uploaded (by Genoveva Galarza; author: Genoveva Galarza):

[mediawiki/extensions/WikiLambda@master] Cleanup: Move isValueTruthy to mixin and add tests

https://gerrit.wikimedia.org/r/997924

Change 997925 had a related patch set uploaded (by Genoveva Galarza; author: Genoveva Galarza):

[mediawiki/extensions/WikiLambda@master] Cleanup: Move selectBestLanguage to mixin and add tests

https://gerrit.wikimedia.org/r/997925

Change 997926 had a related patch set uploaded (by Genoveva Galarza; author: Genoveva Galarza):

[mediawiki/extensions/WikiLambda@master] Cleanup: Separate zobject submodule tests into different files

https://gerrit.wikimedia.org/r/997926

Change 997923 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Cleanup: Rename zobjectTreeUtils mixin and methods

https://gerrit.wikimedia.org/r/997923

Change 997924 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Cleanup: Move isValueTruthy to mixin and add tests

https://gerrit.wikimedia.org/r/997924

Change 997925 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Cleanup: Move selectBestLanguage to mixin and add tests

https://gerrit.wikimedia.org/r/997925

Change 997926 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Cleanup: Separate zobject submodule tests into different files

https://gerrit.wikimedia.org/r/997926