-
Notifications
You must be signed in to change notification settings - Fork 377
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 #8498
base: develop
Are you sure you want to change the base?
Updated blood pressure range #8498
Conversation
✅ Deploy Preview for care-ohc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
CARE Run #3344
Run Properties:
|
Project |
CARE
|
Branch Review |
rithviknishad/fix/blood-pressure-range-and-recordable
|
Run status |
Passed #3344
|
Run duration | 02m 58s |
Commit |
585b645d35: Updated blood pressure range
|
Committer | Rithvik Nishad |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
128
|
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. |
Required Backends
Proposed Changes
@ohcnetwork/care-fe-code-reviewers
Merge Checklist