[SELC-3948] refactor: Workflow Executor and Onboarding state updates #110
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.
List of Changes
Removal of the Dedicated Update Function: Removed the function previously dedicated to updating the onboarding status. This change simplifies the workflow by reducing the number of functions involved in the onboarding process.
Workflow Executor Updates: Modified the workflow executor so that it now returns the next state of the processed onboarding request. This enhancement streamlines the flow of state transitions within the onboarding process.
Orchestration Function Responsibility: Updated the orchestration function to take over the responsibility of updating the request's state. This shift centralizes the control of state transitions to a single function, improving the overall process management.
Motivation and Context
By removing the dedicated function for updating the onboarding status and shifting this responsibility to the orchestration function, we reduce complexity and enhance the maintainability of our codebase. Furthermore, having the workflow executor return the next state directly contributes to a more efficient and clearer state transition mechanism.
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: