Skip to content

Conversation

aditya-07
Copy link
Contributor

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

Fixes #1956

Description

  1. Save local last updated date of resources in a new column in ResourceEntity.
  2. Add local last updated value as a custom DateTimeIndex value to allow search operations.

TODO:
Add tests.
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

  • 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).

@aditya-07 aditya-07 requested review from a team and santosh-pingle as code owners June 8, 2023 11:37
@aditya-07 aditya-07 requested a review from jingtang10 June 8, 2023 11:37
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.

Great stuff Aditya! Left some comments

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.

Superb change! 🚀

@aditya-07 aditya-07 enabled auto-merge (squash) July 21, 2023 08:52
@aditya-07 aditya-07 merged commit 1a23a8a into google:master Jul 21, 2023
MJ1998 pushed a commit to MJ1998/android-fhir that referenced this pull request Jul 22, 2023
…oogle#2030)

* Start saving local lastUpdated to resources for search operations

* Added tests

* Addressed review comments

* Reverted the db encryption flag

* spotless apply

* Review comments: Udpated tests

* Review Comments: Renamed functions to better naming conevntions
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.

Modified Date is needed as sorting parameter for all FHIR resources
3 participants