Closed ndegwamartin closed 1 month ago
@ndegwamartin Thank You for requesting a review from me!
As I only "volunteer" and do not "work for" this project, unfortunately I do not have the "free cycles" to help here.
I trust that some of my colleagues who are working on this project will review this.
Thank You for your understanding!
@MJ1998 ticket created there https://github.com/google/android-fhir/issues/2510
@MJ1998 when you get a chance please assist in merging this
Done
IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #2439
Description
FhirEngine.purge
and modifiedDatabase.purge
since has access modifierinternal
which is reused by the FhirEngine overloads.Database.purge
, wrapped the processing of the resource ids list in a transaction. Either they all pass or it fails.Alternative(s) considered Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
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.