-
Notifications
You must be signed in to change notification settings - Fork 254
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
(test): Fix non-decimal concept having a decimal value #2236
Conversation
Size Change: 0 B Total Size: 16.3 MB ℹ️ View Unchanged
|
I swear to god, this soap note template test case will be the death of me. Its unrelated to this. And its failing for other PRs. |
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.
The failing E2E test(s) are unrelated to these changes!
The SOAP note is really intended to be like a smoke test for "does the most basic possible form actually work" 😁 |
Requirements
Summary
This PR updates the value of
Total Protein
to be a non-decimal value, since its concept doesn't allow decimal values. Although this previously wasn't an issue, the backend now no-longer accepts non-decimal values .Screenshots
Related Issue
Other
This needs to be merged to fix the failing e2e in the PR that bumps the initializer and core version.