-
Notifications
You must be signed in to change notification settings - Fork 54
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
U4X-367 : For tests that don’t have ranges, when printing and approving, they don’t show the test results #43
Conversation
3c3c8c1
to
6cad75e
Compare
if (isError) return <span>Error</span>; | ||
|
||
return ( | ||
<span style={{ marginLeft: "10px" }}>{concept?.units ?? "N/A"}</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it posible to move the styles to the styles file?
<tr key={test} style={{ margin: "10px" }}> | ||
<Checkbox | ||
key={index} | ||
style={{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same to this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CynthiaKamau let me look into this
17129be
to
2aa98c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some quick clean up
package.json
Outdated
"extract-translations": "i18next 'src/**/*.component.tsx' --config ./i18next-parser.config.js", | ||
"coverage": "yarn test -- --coverage", | ||
"prepare": "husky install" | ||
"start": "openmrs develop --backend http://194.163.171.253:8282", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think u can remove this ip
{ | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
residue
Requirements
Summary
This PR fixes non-display of Results that are not Set or GroupMembers
Screenshots
before
after
Related Issue
https://metsprogramme.atlassian.net/browse/U4X-367
Other