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

DPL-894 Convert VueX modules to Pinia stores #1327

Open
11 tasks
seenanair opened this issue Sep 8, 2023 · 0 comments
Open
11 tasks

DPL-894 Convert VueX modules to Pinia stores #1327

seenanair opened this issue Sep 8, 2023 · 0 comments
Labels
Size: L Large - large effort & risk Technical Debt Technical debt Traction Stories related to Traction application Value: 3 Value to the insitute is average

Comments

@seenanair
Copy link
Contributor

seenanair commented Sep 8, 2023

User story
As Pinia has now become the defacto recommended state management library for Vue 3, we are planning to migrate from VueX to Pinia. As part of clean migration, the conversion is planned module by module from VueX to Pinia.

  • Create Pinia directory (stores)
  • For each VueX module
    • Convert to a Pinia store with id same as namespace in Vuex
    • Stores depend on each other will simply import the other store
    • Convert state, getters, action
    • Remove mutation
  • Convert usage inside components
  • Use Options API support of Pinia to maintain the existing code as much as possible

Acceptance criteria
To be considered successful the solution must allow replacement of following Vuex modules:

  • ont/pools
  • ont/runs
  • pacbio/plates
  • pacbio/requests
  • Refactor OntRuns to be OntRunCreate
  • move fetchOntRuns to OntRuns Pinia stores
  • move setInstruments to OntInstruments Pinia store
  • create Pinia store for Pacbio Library Types

Who are the primary contacts for this story
Steve/Seena

Additional context
This story has a non-blocking relationship with:
#1279
#1326
References
[Pinia]
[Migration (https://pinia.vuejs.org/)]
[Migration] (https://pinia.vuejs.org/cookbook/migration-vuex.html)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size: L Large - large effort & risk Technical Debt Technical debt Traction Stories related to Traction application Value: 3 Value to the insitute is average
Projects
None yet
Development

No branches or pull requests

1 participant