-
Notifications
You must be signed in to change notification settings - Fork 246
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 maxSize extension support #1688
Add maxSize extension support #1688
Conversation
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.
Thanks @FikriMilano. 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.
Thanks @FikriMilano for this PR! Added a few comments for review. PTAL and lmk what you think!
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...ure/src/test/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponentsTest.kt
Outdated
Show resolved
Hide resolved
...ure/src/test/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponentsTest.kt
Show resolved
Hide resolved
- Use Kibibytes and Mebibytes. - Add test case when answer size is above limit.
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.
Looks good to me! Will ask @jingtang10 to review quickly and approve!
Awesome, thanks! |
@omarismail94 The CI was having a 120000 timeout error. Let me see if re-triggering it will work. Same with this one #1678 |
The Instrumentation test failed due to the Firebase Android phone timing out. Re-ran it and everything passed! @jingtang10 can you PTAL at the PR and approve? |
@omarismail94 |
I think you do! In the Labels section on the right of the PR page, you can add the |
@omarismail94 |
…size-extension-support
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...ure/src/test/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponentsTest.kt
Outdated
Show resolved
Hide resolved
...ure/src/test/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponentsTest.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
…size-extension-support
2f22623
to
a657118
Compare
@jingtang10 this is ready for another review |
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...capture/src/main/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponents.kt
Outdated
Show resolved
Hide resolved
...ure/src/test/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponentsTest.kt
Outdated
Show resolved
Hide resolved
...ure/src/test/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponentsTest.kt
Outdated
Show resolved
Hide resolved
...ure/src/test/java/com/google/android/fhir/datacapture/MoreQuestionnaireItemComponentsTest.kt
Outdated
Show resolved
Hide resolved
…size-extension-support
IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1675
Description
Allows Questionnaire Item to limit the maximum file size the user can upload
Alternative(s) considered
N/A
Type
Feature
Screenshots (if applicable)
N/A
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.