Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Implements #261 by adding a default method to ProcessPluginDefinition #291

Merged

Conversation

hhund
Copy link
Member

@hhund hhund commented Oct 27, 2021

The new ProcessPluginDefinition.onProcessesDeployed(ApplicationContext pluginApplicationContext, List activeProcesses) method is called after all processes have been deployed and process FHIR resources have been created/updated.

Overriding this Method should allow process plugin implementers to run custom initialization logic (e.g. validating the connection configuration to an external FHIR Server by performing a connection test) after all processes have been deployed and only if a specific process is known to be active.

Resolves #261

Integration tests TBD.

The new ProcessPluginDefinition.onProcessesDeployed(ApplicationContext
pluginApplicationContext, List<String> activeProcesses) method is called
after all processes have been deployed and process FHIR resources have
been created/updated.

Overriding this Method should allow process plugin implementers to run
custom initialization logic (e.g. validating the connection
configuration to an external FHIR Server by performing a connection
test) after all processes have been deployed and only if a specific
process is known to be active.

Resolves #261
@hhund hhund added this to the v0.6.0 milestone Oct 27, 2021
@hhund hhund requested a review from wetret October 27, 2021 11:56
@hhund hhund self-assigned this Oct 27, 2021
issues/261_Mechanism_to_Communicate_Process_Deployment_State
@hhund
Copy link
Member Author

hhund commented Dec 21, 2021

Integration test performed via num-codex/codex-processes-ap1#56

@hhund hhund merged commit 68ba4e5 into develop Dec 21, 2021
@hhund hhund deleted the issues/261_Mechanism_to_Communicate_Process_Deployment_State branch June 30, 2022 15:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Mechanism to Communicate Process Deployment State Back to Process Plugin
2 participants