Skip to content
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

O3-1727: New Test Results Entry form #715

Merged
merged 1 commit into from
Apr 4, 2023

Conversation

gracepotma
Copy link
Contributor

@gracepotma gracepotma commented Apr 4, 2023

Adds a simple but long Test Results Entry form that can be used to manually report recent lab results for some common test types.
The goal is to unblock groups like Faimer who want to be able to record test results, without hooking up a Lab Reporting System.
Known deficiencies:

  • The date/time stamp from the form SUBMISSION will be applied to the Test concept, even if this is not actually reflective of the Test RESULT TIME. We should come back to this in the near future when we work on improving Past Visits/RDE flows.
  • multiCheckbox was not working in the form builder (logged here) so for now I was forced to use Radio where Checkbox would be much more appropriate.

Issue for this PR: https://issues.openmrs.org/browse/O3-1727

Adds a simple but long Test Results Entry form that can be used to manually report recent lab results for some common test types.
@gracepotma gracepotma changed the title New Test Results Entry form O3-1727: New Test Results Entry form Apr 4, 2023
@ibacher ibacher merged commit c1e8824 into main Apr 4, 2023
@ibacher ibacher deleted the gracepotma-TestResultsEntryFormv2 branch April 4, 2023 15:56
@rbuisson
Copy link
Contributor

rbuisson commented Apr 5, 2023

Looks like this file is missing an extension. I'm guessing .json 😉

@gracepotma
Copy link
Contributor Author

Yup, how embarrassing. Thanks @rbuisson for the catch. Fixed in mainline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants