-
Notifications
You must be signed in to change notification settings - Fork 293
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
Update questionnaire buttons in xml fragment #2384
Update questionnaire buttons in xml fragment #2384
Conversation
d56b1c7
to
29150cc
Compare
@MJ1998 Update has been made |
Thanks. Please add a description to the solution! |
@MJ1998 I did further testing and noted that if the |
You are right! My IDE was throwing some error. After invalidate-restart its good. |
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
@MJ1998 Kindly prompt kokoro |
* Add fhirpath supplements * Add unit tests * Update datacapture/src/main/java/com/google/android/fhir/datacapture/fhirpath/ExpressionEvaluator.kt Co-authored-by: Jing Tang <[email protected]> * Update datacapture/src/main/java/com/google/android/fhir/datacapture/fhirpath/ExpressionEvaluator.kt Co-authored-by: Jing Tang <[email protected]> * Resolve feedback and test issues * Fix compile issue * Update datacapture/src/test/java/com/google/android/fhir/datacapture/fhirpath/ExpressionEvaluatorTest.kt Co-authored-by: Jing Tang <[email protected]> * Rename tests --------- Co-authored-by: Jing Tang <[email protected]>
b76cef5
to
be1bcb2
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
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 we can remove the visibility attribute change, as it get handled through the flags in the configuration.
Co-authored-by: santosh-pingle <[email protected]>
Co-authored-by: santosh-pingle <[email protected]>
Build fails on spotless apply. |
Head branch was pushed to by a user without write access
70e9cbe
to
c2c78ad
Compare
IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #2353
Description
The
submit
button has been made configurable and the layout can be changed based on one's use case. The rest of the buttonsprevious
,next
andreview
have also been made configurable as well based on the designs in the issue above.Clear and concise code change description.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one:Feature
Screenshots (if applicable)
Checklist
./gradlew spotlessApply
and./gradlew spotlessCheck
to check my code follows the style guide of this project../gradlew check
and./gradlew connectedCheck
to test my changes locally.