-
Notifications
You must be signed in to change notification settings - Fork 305
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
Engine codelab #2059
Engine codelab #2059
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.
Can you make sure the README.md file is consistent with the one on the developer website?
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'm approving this assuming the last commit resolved my earlier comment on making the readme file the same as the dev site
done. thanks |
🤘🚀 |
* distinct name for onetime new stuff Fix the version collision in Knowledge Manager (#2043) * Url is not a unique identifier for the library and should be used together with version. Made necessary changes to reflect that * Truncate the input test file to fit the memory. Use the first library with the given url if the version wasn't explicitly provided * Refactored text overflow in dropdown (#1987) * Refactored text overflow in dropdown * Removed unnecessary attributes from drop down list item xml * Update datacapture/src/main/res/layout/drop_down_list_item.xml --------- Co-authored-by: Jing Tang <jingtang@google.com> * Spotless apply --------- Co-authored-by: Omar Ismail <44980219+omarismail94@users.noreply.github.com> Co-authored-by: khyativyasargus <khyativ@argusoft.com> Co-authored-by: Jing Tang <jingtang@google.com> Support basic HTTP authentication (#2038) * Create a more generic and consistent way of providing authorization header * Update engine/src/main/java/com/google/android/fhir/sync/HttpAuthenticator.kt Co-authored-by: Omar Ismail <44980219+omarismail94@users.noreply.github.com> * Update doc for HttpAuthenticator --------- Co-authored-by: Omar Ismail <44980219+omarismail94@users.noreply.github.com> Engine codelab (#2059) * Initial version * make app work * add update function * Run spotless apply * Remove unnecessary header in FHIR Engine codelab README.md --------- Co-authored-by: omarismail <omarismail@google.com> Simplify codelab code (#2060) * Initial version * make app work * add update function * Run spotless apply * Remove unnecessary header in FHIR Engine codelab README.md * Simplify engine codelab code * Update codelabs/engine/app/src/main/java/com/google/android/fhir/codelabs/engine/DownloadWorkManagerImpl.kt Co-authored-by: Omar Ismail <44980219+omarismail94@users.noreply.github.com> --------- Co-authored-by: omarismail <omarismail@google.com> Co-authored-by: Omar Ismail <44980219+omarismail94@users.noreply.github.com> Engine codelab (#2061) * Initial version * make app work * add update function * Run spotless apply * Remove unnecessary header in FHIR Engine codelab README.md * Simplify engine codelab code * Update codelabs/engine/app/src/main/java/com/google/android/fhir/codelabs/engine/DownloadWorkManagerImpl.kt Co-authored-by: Omar Ismail <44980219+omarismail94@users.noreply.github.com> * Update engine codelab * Run spotless --------- Co-authored-by: omarismail <omarismail@google.com> Co-authored-by: Omar Ismail <44980219+omarismail94@users.noreply.github.com> * address comments
IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Description
FHIR Engine codelab initial version
Alternative(s) considered
NA
Type
Documentation
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.