-
Notifications
You must be signed in to change notification settings - Fork 475
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated blood pressure range and remove hyphen for infusion "nor-adrenaline" #8498
Updated blood pressure range and remove hyphen for infusion "nor-adrenaline" #8498
Conversation
✅ Deploy Preview for care-ohc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
CARE Run #3412
Run Properties:
|
Project |
CARE
|
Branch Review |
rithviknishad/fix/blood-pressure-range-and-recordable
|
Run status |
Passed #3412
|
Run duration | 03m 09s |
Commit |
0f1ba56027: Updated blood pressure range and remove hyphen for infusion "nor-adrenaline"
|
Committer | Rithvik Nishad |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
120
|
View all changes introduced in this branch ↗︎ |
@rithviknishad the ux assumes that not measurable is a very common scenario; do we need it? How valuable is not measurable information? Can't that be part of the notes? |
The layout of the short vitals form (with text box and not slider) looks very bad. Can we have "Not measurable" come below the text field? The text spaces for systolic and diastolic must come together as "____/_____"- having the checkbox come in between the field is not ideal. |
"Not measurable" is not a very common scenario. design to be improved to have the option come up below BP fields, not above |
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
|
LGTM |
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@rithviknishad Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Required Backends
Proposed Changes
@ohcnetwork/care-fe-code-reviewers
Merge Checklist