Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Refactor Imaging module to use react query instead of redux #2340

Closed
jackcmeyer opened this issue Aug 27, 2020 · 0 comments · Fixed by #2355
Closed

Refactor Imaging module to use react query instead of redux #2340

jackcmeyer opened this issue Aug 27, 2020 · 0 comments · Fixed by #2355
Assignees
@jackcmeyer
Labels
🚀enhancement an issue/pull request that adds a feature to the application imaging in progress indicates that issue/pull request is currently being worked on LOE - large indicates that the level of effort to complete issue is large
Milestone
v2.0

Comments

@jackcmeyer
Copy link
Member

🚀 Feature Proposal

Refactor the imaging module to to use react query instead of redux

Need 4 hooks

  • useSearchImagingRequest
  • useRequestImaging
  • useCancelLabel
  • useImageRequest

Refactor the code to use these new hooks

Motivation

Remove redux

Technical Notes

use incidents module for inspiration
remove redux after completion
@jackcmeyer jackcmeyer transferred this issue from HospitalRun/hospitalrun Aug 27, 2020
@jackcmeyer jackcmeyer added help wanted indicates that an issue is open for contributions imaging LOE - large indicates that the level of effort to complete issue is large 🚀enhancement an issue/pull request that adds a feature to the application labels Aug 27, 2020
@jackcmeyer jackcmeyer added this to the v2.0 milestone Aug 27, 2020
@jackcmeyer jackcmeyer self-assigned this Sep 1, 2020
@jackcmeyer jackcmeyer mentioned this issue Sep 1, 2020
Merged
@blestab blestab added in progress indicates that issue/pull request is currently being worked on and removed help wanted indicates that an issue is open for contributions labels Sep 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jackcmeyer jackcmeyer

Labels
🚀enhancement an issue/pull request that adds a feature to the application imaging in progress indicates that issue/pull request is currently being worked on LOE - large indicates that the level of effort to complete issue is large
Projects
None yet
Milestone
v2.0
2 participants
@blestab @jackcmeyer