-
Notifications
You must be signed in to change notification settings - Fork 363
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
(feat) Add mental health assessment form #843
Conversation
0e71ade
to
0bd2730
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.
LGTM! Thanks, Nethmi.
@ibacher does the |
@denniskigen They definitely should pass or else there's very likely something broken. |
In this case, the failure message is: "Form Encounter type Mental Health Assessment could not be found. Please ensure that this encountertype is either loaded by Iniz or loaded in the system before Iniz runs." So that's a pretty obvious "we're missing necessary metadata to make this form usable" issue. |
@ibacher that should be included in this diff, right? Could you take care of that, @NethmiRodrigo? |
Yeah, if we're requiring an EncounterType it should be in the metadata. |
9b1a39e
to
aba325f
Compare
Any idea why this might be happening @ibacher @denniskigen causing the form builder e2e to fail? |
I've just rerun the tests, @NethmiRodrigo. Should be ok now. |
This PR adds the PHQ-9 Screening Tool form.
https://openmrs.atlassian.net/browse/O3-3717