Skip to content
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

Simplify codelab code #2060

Merged
merged 11 commits into from
Jul 1, 2023
Merged

Simplify codelab code #2060

merged 11 commits into from
Jul 1, 2023

Conversation

jingtang10
Copy link
Collaborator

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).

Description
Simplify codelab code

Alternative(s) considered
NA

Type
Choose one: Code health

Screenshots (if applicable)

Checklist

  • I have read and acknowledged the Code of conduct.
  • I have read the Contributing page.
  • I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
  • I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
  • I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
  • I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
  • I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

@jingtang10 jingtang10 requested review from a team and santosh-pingle as code owners July 1, 2023 18:06
@jingtang10 jingtang10 requested a review from aditya-07 July 1, 2023 18:06
Copy link
Contributor

@omarismail94 omarismail94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, much simplified!!

…labs/engine/DownloadWorkManagerImpl.kt

Co-authored-by: Omar Ismail <[email protected]>
@jingtang10 jingtang10 enabled auto-merge (squash) July 1, 2023 19:46
@jingtang10 jingtang10 merged commit 2251940 into google:master Jul 1, 2023
2 of 3 checks passed
omarismail94 added a commit that referenced this pull request Jul 4, 2023
* 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 <[email protected]>

* Spotless apply

---------

Co-authored-by: Omar Ismail <[email protected]>
Co-authored-by: khyativyasargus <[email protected]>
Co-authored-by: Jing Tang <[email protected]>

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 <[email protected]>

* Update doc for HttpAuthenticator

---------

Co-authored-by: Omar Ismail <[email protected]>

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 <[email protected]>

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 <[email protected]>

---------

Co-authored-by: omarismail <[email protected]>
Co-authored-by: Omar Ismail <[email protected]>

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 <[email protected]>

* Update engine codelab

* Run spotless

---------

Co-authored-by: omarismail <[email protected]>
Co-authored-by: Omar Ismail <[email protected]>

* address comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

None yet

2 participants