Closed chriswhong closed 2 years ago
DeployModal
and SplashModal
are renamed AutomationCommitModal
and AutomationSplashModal
API_PREVIEW_SUCCESS
from workflowState, this was used to set workflow.dataset
to be the active dataset but was causing problems when a dataset didn't have a transform. Instead ExistingAutomationEditor
now checks for the transform, and if one doesn't exist, it dispatches the new setWorkflowDataset
action and adds the template code.CommitBar
component is now used in all editors
Refactors the workflow creation and editing UI, Closes #616
Automation happens via two components
NewAutomationEditor
andExistingAutomationEditor
which are children of their respective routes. Both of these renderWorkflowPage
, which incorporates the sameEditorLayout
we are using in the static dataset editor UI.WorkflowPage
then rendersWorkflowEditor
, which contains the code blocks, new sidebar, etc.We have replaced "workflow" in the UI, but it is still present in the codebase. Should be seek to change all references to "workflow" into "automation"? This would carry over into the backend as well, and may not be worth the effort since workflow is still a thing in the behind the scenes.
In addition to reviewing the code, please pull this down and use it. Try to go through both creation of an automated dataset and editing the automation of an existing dataset.
TODO:
Found during review: