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

[Event Management] Update the CarePlan Closure configs to define the PlanDefs to closure separately. #2372

Closed
5 tasks
dubdabasoduba opened this issue May 23, 2023 · 0 comments · Fixed by #2407
Closed
5 tasks
Assignees
Labels
App Configuration Bug Report Something isn't working as expected Event Management

Comments

@dubdabasoduba
Copy link
Member

dubdabasoduba commented May 23, 2023

Description of the Issue

  • Currently, the closure of events depends on the definition of PlanDefs in the "questionnaire": {} configuration in the ON-CLICK action.
  • This configuration is also used by the Workflow start functionality. This setup creates a risk of closing wrong workflows or opening unnecessary workflows for clients.

Example to Highlight the Above Risk

  • Suppose the PlanDef configuration has 10 PlanDefinitions defined, where 4 are required to open new workflows and 6 are required to close workflows.
  • Currently, when the event condition is met, the Workflow closure code will close all 10 Careplans referenced by the PlanDefs defined in the array, regardless of whether they should be closed or not. However, all 10 PlanDefs have the potential to open new workflows if the conditions in the actions are met.

High-Level Implementation

  • Move the Event Management PlanDefinition configuration to the eventWorkflows configuration.
  • Ensure that the PlanDefinitions defined in eventWorkflows are executed only when the required conditions are met.

Acceptance Criteria

  • All Event Management PlanDefinitions must be defined in the eventWorkflows configuration.
  • All Event Management PlanDefinitions can only be closed if their defined condition is true.
  • Each condition should target particular PlanDefinition(s) to be closed whenever the condition is met

The eventWorkflows will be created here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Configuration Bug Report Something isn't working as expected Event Management
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants