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

Fix allergies bug #1040

Merged
merged 145 commits into from
Mar 31, 2017
Merged

Fix allergies bug #1040

merged 145 commits into from
Mar 31, 2017

Conversation

adeolabadmus
Copy link
Contributor

Fixes #994
This PR revamps how allergies work in the app.

Changes proposed in this pull request:

  • Rendered allergy modal into application modal outlet, rather than patient-summary
  • Created allergy pod(controller & template)
  • Bubbled add & edit allergy actions to current editController
  • Created allergy-actions mixin
  • Added validation to allergy model(Previously, allergies could be saved without filling name field)
  • Added acceptance test for allergy on visit page.

cc @HospitalRun/core-maintainers

Chima1707 and others added 30 commits January 26, 2017 16:23
Copy link
Member

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks for the PR @adeolabadmus!

@jkleinsc jkleinsc merged commit 7d961ec into HospitalRun:master Mar 31, 2017
@jkleinsc jkleinsc mentioned this pull request Mar 31, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants