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

Add requesting clinician to Labs #450

Merged
merged 1 commit into from
May 13, 2016
Merged

Add requesting clinician to Labs #450

merged 1 commit into from
May 13, 2016

Conversation

clettenberg
Copy link
Contributor

Fixes #364 .

Changes proposed in this pull request:

edit
completed
requested

cc @HospitalRun/core-maintainers

@clettenberg clettenberg mentioned this pull request May 10, 2016
Closed
jkleinsc
jkleinsc reviewed May 12, 2016
View reviewed changes
app/styles/components/_labs.scss
@@ -0,0 +1,2 @@
.labs-row .notes { max-width: 300px; }
.completed-labs-row .notes { max-width: 300px; }
Copy link
Member

Choose a reason for hiding this comment

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

@cacqw7 saw comment as #437 - not sure why we need two different classes here. @jglovier any opinion on using two different classes for completed vs in-progress?

@clettenberg clettenberg force-pushed the lab-request-clinician branch from f5a5895 to 7a502d5 Compare May 13, 2016 01:00
@clettenberg clettenberg force-pushed the lab-request-clinician branch from 7a502d5 to 60138f5 Compare May 13, 2016 01:08
@clettenberg
Copy link
Contributor Author

@jkleinsc I removed those classes 👍

@jkleinsc jkleinsc merged commit 60138f5 into HospitalRun:master May 13, 2016
jkleinsc added a commit that referenced this pull request May 13, 2016
@jkleinsc
Add requesting clinician to Labs
16af705 
Merging PR #450
Closes #450
Fixes #364
@jkleinsc
Copy link
Member

Thanks @cacqw7. Merging this one too.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@clettenberg @jkleinsc