mbrown1413 / vgloss

A web-based media organization program.
0 stars 1 forks source link

Remove Vuex #24

Closed mbrown1413 closed 3 years ago

mbrown1413 commented 4 years ago

Instead of using Vuex, we should just have a module "state.js" that tracks all global state and houses functions for manipulating it. Use Vue.observable() to make state observable. In the future we can have global state for loading, use this central place to limit in-flight queries, cache lookups centrally, etc.

My only hesitation on getting rid of Vuex was the ability to view the current state via devtools. However, in webpack you can expose internal modules using output.LibraryTarget

Of course, we need to document where this data is exposed in a section about development and debugging.

mbrown1413 commented 3 years ago

Done with commit 7f9daeeebbf51c03d443b103f10157862ff52f4f "Remove vuex". Debug info is exposed via document._debug (see main.js).