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.

Sign up for GitHub

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

Jump to bottom

UI: Move Namespaces to use RTK #2447

Closed
markphelps opened this issue Nov 26, 2023 · 2 comments · Fixed by #2472
Closed

UI: Move Namespaces to use RTK #2447

markphelps opened this issue Nov 26, 2023 · 2 comments · Fixed by #2472
Assignees
@erka @markphelps
Labels
good first issue help wanted Halp ui

Comments

@markphelps
Copy link
Collaborator

Like in #2441

Should try to move namespaces over from using the existing state management with dispatch, asyncThunks, etc to using RTK

https://redux-toolkit.js.org/rtk-query/overview
@markphelps markphelps self-assigned this Nov 27, 2023
@erka
Copy link
Collaborator

erka commented Nov 29, 2023

@markphelps will you do it? Or can I do it?

@markphelps
Copy link
Collaborator Author

yah i wont be able to get to it until next week, so if you are available/interested that would be great!

erka added a commit to erka/flipt that referenced this issue Nov 29, 2023
@erka
refactor(ui): move namespaces to use redux rtk
7d7d84a 
fixes: flipt-io#2447
@erka erka mentioned this issue Nov 29, 2023
Merged
erka added a commit to erka/flipt that referenced this issue Nov 30, 2023
@erka
refactor(ui): move namespaces to use redux rtk
8941d24 
fixes: flipt-io#2447
markphelps added a commit that referenced this issue Dec 2, 2023
@erka @markphelps
refactor(ui): move namespaces to use redux rtk (#2472)
c75ed08 
fixes: #2447

Co-authored-by: Mark Phelps <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erka erka

@markphelps markphelps

Labels
good first issue help wanted Halp ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor(ui): move namespaces to use redux rtk erka/flipt
2 participants
@erka @markphelps