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

Care Plan Note should not be required #2127

Closed
jackcmeyer opened this issue Jun 16, 2020 · 2 comments · Fixed by #2158
Closed

Care Plan Note should not be required #2127

jackcmeyer opened this issue Jun 16, 2020 · 2 comments · Fixed by #2158
Assignees
@anyapawar
Labels
🐛bug issue/pull request that documents/fixes a bug in progress indicates that issue/pull request is currently being worked on patients issue/pull request that interacts with patients module
Milestone
v2.0

Comments

@jackcmeyer
Copy link
Member

🐛 Bug Report

When creating a care plan, the note field should not be required.

To Reproduce

  1. navigate to patients -> care plan -> add care plan
  2. fill in the required fields
  3. note should not be required

Expected behavior

  1. note should not be required

Technical notes

  • remote validation from redux thunk
  • remove required asterisk
@jackcmeyer jackcmeyer added 🐛bug issue/pull request that documents/fixes a bug help wanted indicates that an issue is open for contributions good first issue indicates an issue is good for a first time contributor patients issue/pull request that interacts with patients module labels Jun 16, 2020
@jackcmeyer jackcmeyer added this to the v2.0 milestone Jun 16, 2020
@anyapawar
Copy link
Contributor

Would like to take this one!

@matteovivona
Copy link
Contributor

Assigned to @anyapawar

@matteovivona matteovivona added in progress indicates that issue/pull request is currently being worked on and removed good first issue indicates an issue is good for a first time contributor help wanted indicates that an issue is open for contributions labels Jun 16, 2020
reidmeyer added a commit to reidmeyer/hospitalrun-frontend that referenced this issue Jun 19, 2020
@reidmeyer
fix: issue 2127. Care Plan Note Not Required
8843070 
Worked with WUSTL Group (Anya Pawar, Samika Kikkeri, Fadel Alkilani). First issue.

fix HospitalRun#2127
This was referenced Jun 19, 2020
Closed
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@anyapawar anyapawar

Labels
🐛bug issue/pull request that documents/fixes a bug in progress indicates that issue/pull request is currently being worked on patients issue/pull request that interacts with patients module
Projects
None yet
Milestone
v2.0
3 participants
@matteovivona @jackcmeyer @anyapawar