Externalizes more subscription parameters #783
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@Jin66 if you update your branch, then I'll have a look |
Allows : - to parametrize the polling interval of subscription - to enable the setting to queue the subscription jobs immediately. Following the change in hapifhir/hapi-fhir/pull/6395
Jin66
force-pushed
the
allows_to_parametrize_subscription_polling
branch
from
6aecec1 to
63badd8
Compare
jkiddo
approved these changes
Mar 2, 2025
|
Thank you ;) |
Salaton
added a commit
to savannahghi/hapi-fhir-jpaserver-starter
that referenced
this pull request
Mar 5, 2025
* Bump to HAPI 7.7.7-SNAPSHOT * Update HAPI + match config changes * Bump HAPI to 7.7.10-SNAPSHOT * Bump parent to 7.7.14-SNAPSHOT * Bump parent version + fix CQL test See diff in https://github.com/cqframework/clinical-reasoning/pull/579/files * Bump parent to 7.7.18-SNAPSHOT fix CR breakage * use URI constructor instead of string concatenation, to avoid having static code analysis tool complain about "Concatenating user-controlled input into a URL" security issue * Add support for Database Partition Mode * Add spotless-check * Apply spotless * Apply spotless * Revert "Apply spotless" This reverts commit 8b77aa0. * Test fix * Spotless * Helm chart dependency updates and support for extraVolumes and -Mounts (hapifhir#766) * Helm chart updates * set target branch to master for chart testing * Bump to release * Expose expansion properties as configuration * Externalizes more subscription parameters (hapifhir#783) * Externalizes more subscription parameters Allows : - to parametrize the polling interval of subscription - to enable the setting to queue the subscription jobs immediately. Following the change in hapifhir/hapi-fhir/pull/6395 * Fix formatting * feat: add a github actions deploy workflow * chore: deploy hapi fhir staging and production instance * chore: use UUIDs for resource IDs * feat: updating FHIR version from R4 to R5 --------- Co-authored-by: dotasek.dev <dotasek.dev@gmail.com> Co-authored-by: dotasek <david.otasek@smilecdr.com> Co-authored-by: Joel Schneider <jschneid@nmdp.org> Co-authored-by: James Agnew <jamesagnew@gmail.com> Co-authored-by: chgl <chgl@users.noreply.github.com> Co-authored-by: Adam Stevenson <stevenson_adam@yahoo.com> Co-authored-by: Yoann Isaac <Jin66@users.noreply.github.com> Co-authored-by: Kennedy Omondi <kennankole@gmail.com>
Salaton
added a commit
to savannahghi/hapi-fhir-jpaserver-starter
that referenced
this pull request
Mar 5, 2025
* Bump to HAPI 7.7.7-SNAPSHOT * Update HAPI + match config changes * Bump HAPI to 7.7.10-SNAPSHOT * Bump parent to 7.7.14-SNAPSHOT * Bump parent version + fix CQL test See diff in https://github.com/cqframework/clinical-reasoning/pull/579/files * Bump parent to 7.7.18-SNAPSHOT fix CR breakage * use URI constructor instead of string concatenation, to avoid having static code analysis tool complain about "Concatenating user-controlled input into a URL" security issue * Add support for Database Partition Mode * Add spotless-check * Apply spotless * Apply spotless * Revert "Apply spotless" This reverts commit 8b77aa0. * Test fix * Spotless * Helm chart dependency updates and support for extraVolumes and -Mounts (hapifhir#766) * Helm chart updates * set target branch to master for chart testing * Bump to release * Expose expansion properties as configuration * Externalizes more subscription parameters (hapifhir#783) * Externalizes more subscription parameters Allows : - to parametrize the polling interval of subscription - to enable the setting to queue the subscription jobs immediately. Following the change in hapifhir/hapi-fhir/pull/6395 * Fix formatting * feat: add a github actions deploy workflow * chore: deploy hapi fhir staging and production instance * chore: use UUIDs for resource IDs * feat: updating FHIR version from R4 to R5 --------- Co-authored-by: dotasek.dev <dotasek.dev@gmail.com> Co-authored-by: dotasek <david.otasek@smilecdr.com> Co-authored-by: Joel Schneider <jschneid@nmdp.org> Co-authored-by: James Agnew <jamesagnew@gmail.com> Co-authored-by: chgl <chgl@users.noreply.github.com> Co-authored-by: Adam Stevenson <stevenson_adam@yahoo.com> Co-authored-by: Yoann Isaac <Jin66@users.noreply.github.com> Co-authored-by: Kennedy Omondi <kennankole@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Allows :
Following the change in #hapifhir/hapi-fhir/pull/6395