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

refactor(patient): decouple patient validator from redux code #2471

Merged

Conversation

giulianovarriale
Copy link
Contributor

This change is an intermediate step to solve #2378, #2382.

Changes proposed in this pull request:

  • Create a shared patient validator to be able to use on different hooks

@jsf-clabot
Copy link

jsf-clabot commented Nov 2, 2020

CLA assistant check
All committers have signed the CLA.

@gitpod-io
Copy link

gitpod-io bot commented Nov 2, 2020

@vercel
Copy link

vercel bot commented Nov 2, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hospitalrun/hospitalrun-frontend/geklpke0m
✅ Preview: https://hospitalrun-frontend-git-refactor-patient-validator.hospitalrun.vercel.app

@matteovivona matteovivona added the in progress indicates that issue/pull request is currently being worked on label Nov 6, 2020
@matteovivona matteovivona added this to In progress in Version 2.0 via automation Nov 6, 2020
@matteovivona matteovivona added this to the v2.0 milestone Nov 6, 2020
@matteovivona matteovivona self-requested a review November 6, 2020 20:41
@jackcmeyer jackcmeyer changed the title refactor(patient-validator): decouple patient validator from redux code refactor(patient): decouple patient validator from redux code Nov 10, 2020
@jackcmeyer jackcmeyer merged commit b753d90 into HospitalRun:master Nov 10, 2020
Version 2.0 automation moved this from In progress to Done Nov 10, 2020
@giulianovarriale giulianovarriale deleted the refactor/patient-validator branch November 10, 2020 20:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
in progress indicates that issue/pull request is currently being worked on
Projects
Version 2.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants