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

[ServerSide] Sync the OpenSRP FHIR Gateway repo with the upstream Google FHIR Gateway repo #29

Merged
merged 11 commits into from
May 15, 2023

Conversation

rehammuzzamil
Copy link

vivekmittal07 and others added 7 commits April 18, 2023 12:09
* Define access decision preprocess stage to mutate the request params

* Documentation for request mutation change. Added test cases.

* Removed static identifier from request mutation method
Add default scan paths in the exec application
* supporting gzip for client and server data transfer

* adding comments for test case

* addressing-review-comments
@ekigamba
Copy link

@rehammuzzamil Can you update this branch? It's out of date and the diff's are wrong. For the conflicts in the Dockerfile and pom.xml files, kindly prefer what we have on main as this contains the latest version and docker build fixes

ekigamba
ekigamba previously approved these changes May 15, 2023
@ndegwamartin
Copy link

@rehammuzzamil we need to update the various pom files to version 0.1.19-beta

@rehammuzzamil
Copy link
Author

@rehammuzzamil we need to update the various pom files to version 0.1.19-beta

Done @ndegwamartin

@rehammuzzamil rehammuzzamil requested a review from ekigamba May 15, 2023 10:52
@ndegwamartin
Copy link

We can do the migration to align with the refactors from upstream as part of a new ticket cc @dubdabasoduba @rehammuzzamil @ekigamba

@ndegwamartin ndegwamartin merged commit 208f0fc into main May 15, 2023
@ndegwamartin ndegwamartin deleted the 2340-copy-of-main branch May 15, 2023 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants