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

Add Diagnosis Modal should mark required fields #1926

Closed
jackcmeyer opened this issue Mar 19, 2020 · 2 comments 路 Fixed by #1936
Closed

Add Diagnosis Modal should mark required fields #1926

jackcmeyer opened this issue Mar 19, 2020 · 2 comments 路 Fixed by #1936
Assignees
@nwmandel
Labels
馃殌enhancement an issue/pull request that adds a feature to the application good first issue indicates an issue is good for a first time contributor in progress indicates that issue/pull request is currently being worked on patients issue/pull request that interacts with patients module
Projects
Version 2.0
Milestone
v2.0

Comments

@jackcmeyer
Copy link
Member

馃殌 Feature Proposal

The diagnosis name field and diagnosis date field should be marked with a * to indicate that it is required.

Motivation

Better use experience.
@jackcmeyer jackcmeyer transferred this issue from HospitalRun/hospitalrun Mar 19, 2020
@jackcmeyer jackcmeyer added this to the v2.0 milestone Mar 19, 2020
@jackcmeyer jackcmeyer added this to To do in Version 2.0 via automation Mar 19, 2020
@jackcmeyer jackcmeyer added 馃殌enhancement an issue/pull request that adds a feature to the application good first issue indicates an issue is good for a first time contributor help wanted indicates that an issue is open for contributions patients issue/pull request that interacts with patients module labels Mar 19, 2020
@nwmandel
Copy link
Contributor

Hi @jackcmeyer, mind if I tackle this issue? 馃榾

@jackcmeyer
Copy link
Member Author

@nwmandel go ahead!

@jackcmeyer jackcmeyer 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 Mar 21, 2020
nwmandel added a commit to nwmandel/hospitalrun-frontend that referenced this issue Mar 21, 2020
@nwmandel
fix(adddiagnosismodal and datepickerwithlabelformgroups): required field
b4100f5 
Added required fields to diagnosis name and date

fix HospitalRun#1926
@nwmandel nwmandel mentioned this issue Mar 21, 2020
Merged
Version 2.0 automation moved this from To do to Done Mar 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nwmandel nwmandel

Labels
馃殌enhancement an issue/pull request that adds a feature to the application good first issue indicates an issue is good for a first time contributor in progress indicates that issue/pull request is currently being worked on patients issue/pull request that interacts with patients module
Projects
Version 2.0
  
Done
Milestone
v2.0
Development

Successfully merging a pull request may close this issue.

fix(patients): add required field label for diagnosis nwmandel/hospitalrun-frontend
2 participants
@jackcmeyer @nwmandel