-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add helper function to ignore hidden questions #325
Conversation
efsity-cli/src/main/java/org/smartregister/command/QuestionnaireResponseGeneratorCommand.java
Outdated
Show resolved
Hide resolved
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.
@sharon2719 the main reason we are ignoring these questions is because they are "ideally" pre-populated by something else. Is it possible to trigger whatever should do this pre-population so that these answers are also in the response?
Most of the prepopulation is done by the rules that re set in the profiles and registers. Probably we can come up with a way that will do. Some are also linked to other questionnaires ie commodity |
Can we maybe scope a separate issue for this? Something like if you choose to ignore these hidden questions the response will come up empty, but if you'd like them populated then you have the option of passing in the profile or register etc? |
…ng into ignore-hidden-questions
Sure this is very okay |
IMPORTANT: Where possible all PRs must be linked to a Github issue
Closes #319
Engineer Checklist
./gradlew spotlessApply
to check my code follows the project's style guide./gradlew clean assemble
and run the efsity jar to verify my change fixes the issue and does not break the application