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

create useUpdatePatient hook and remove related redux code #2382

Open
jackcmeyer opened this issue Sep 9, 2020 · 3 comments
Open

create useUpdatePatient hook and remove related redux code #2382

jackcmeyer opened this issue Sep 9, 2020 · 3 comments
Assignees
@giulianovarriale
Labels
馃悰bug issue/pull request that documents/fixes a bug in progress indicates that issue/pull request is currently being worked on
Projects
Version 2.0
Milestone
v2.0

Comments

@jackcmeyer
Copy link
Member

馃殌 Feature Proposal

Create a useUpdatePatient hook which will update a patient and remove all related redux code for getting a patient by id.

Motivation

  • remove redux

Technical Notes

  • start using the useUpdatePatient hook for editing a patient
  • remove redux code and related tests
@jackcmeyer jackcmeyer added 馃殌enhancement an issue/pull request that adds a feature to the application help wanted indicates that an issue is open for contributions patients issue/pull request that interacts with patients module LOE - small indicates that the level of effort to complete issue is small (i.e changing the color of a button) labels Sep 9, 2020
@jackcmeyer jackcmeyer added this to the v2.0 milestone Sep 9, 2020
@jackcmeyer jackcmeyer added this to To do in Version 2.0 via automation Sep 9, 2020
@maciekglowacki
Copy link

Hi, @jackcmeyer. Could I take this issue? :D

@matteovivona
Copy link
Contributor

Absolutely yes. Good work!

@matteovivona matteovivona 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 9, 2020
@matteovivona matteovivona added 馃悰bug issue/pull request that documents/fixes a bug help wanted indicates that an issue is open for contributions hacktoberfest and removed LOE - small indicates that the level of effort to complete issue is small (i.e changing the color of a button) in progress indicates that issue/pull request is currently being worked on patients issue/pull request that interacts with patients module 馃殌enhancement an issue/pull request that adds a feature to the application help wanted indicates that an issue is open for contributions labels Oct 7, 2020
@giulianovarriale
Copy link
Contributor

I see this story was picked up long time ago. Can I take this one?

This was referenced Oct 31, 2020
Closed
Closed
@matteovivona matteovivona added hacktoberfest in progress indicates that issue/pull request is currently being worked on labels Nov 1, 2020
@giulianovarriale giulianovarriale mentioned this issue Nov 2, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@giulianovarriale giulianovarriale

Labels
馃悰bug issue/pull request that documents/fixes a bug in progress indicates that issue/pull request is currently being worked on
Projects
Version 2.0
  
To do
Milestone
v2.0
Development

Successfully merging a pull request may close this issue.

refactor(edit patient): use react-query when editing a patient giulianovarriale/hospitalrun-frontend
4 participants
@giulianovarriale @matteovivona @jackcmeyer @maciekglowacki