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-979 Convert VueX module pacbio/requests to Pinia store #1397

Open
4 tasks
seenanair opened this issue Nov 3, 2023 · 1 comment · May be fixed by #2138
Open
4 tasks

DPL-979 Convert VueX module pacbio/requests to Pinia store #1397

seenanair opened this issue Nov 3, 2023 · 1 comment · May be fixed by #2138
Assignees
Labels
On Hold On Hold for further information Pinia Pinia Size: M Medium - medium 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 Nov 3, 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.

  • Convert pacbio VueX module runs in store/traction/pacbio/requests to Pinia store
    • Convert to a Pinia store with id same as namespace in Vuex
    • 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 pacbio Vuex module to Pinia stores

  • move setRequests and updateRequests to pacbioRequests Pinia store
  • move createRequestPayload to utility function
  • move mutations and state to pacbioRequests Pinia store
  • remove usage of PacbioSampleFactory and replace with PacbioRequestFactory

Who are the primary contacts for this story
Steve/Seena

Additional context
This story has a non-blocking relationship with:

References
[Pinia]
[Migration (https://pinia.vuejs.org/)]
[Migration] (https://pinia.vuejs.org/cookbook/migration-vuex.html)

@seenanair seenanair self-assigned this Nov 3, 2023
@seenanair seenanair added Size: M Medium - medium effort & risk Traction Stories related to Traction application Technical Debt Technical debt Value: 3 Value to the insitute is average labels Nov 3, 2023
@seenanair
Copy link
Contributor Author

@stevieing Added new Pinia story

@seenanair seenanair changed the title DPL-978 Convert VueX module pacbio/requests to Pinia store DPL-979 Convert VueX module pacbio/requests to Pinia store Nov 3, 2023
@seenanair seenanair removed their assignment Dec 16, 2024
@stevieing stevieing added Pinia Pinia Size: S Small - low effort & risk Size: M Medium - medium effort & risk and removed Size: M Medium - medium effort & risk Size: S Small - low effort & risk labels Dec 16, 2024
@stevieing stevieing added the On Hold On Hold for further information label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Hold On Hold for further information Pinia Pinia Size: M Medium - medium effort & risk Technical Debt Technical debt Traction Stories related to Traction application Value: 3 Value to the insitute is average
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants