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

Fix confirmation modal appearing when adding a transformation while creating or saving a connection #13426

Merged
merged 4 commits into from
Jun 10, 2022

Conversation

edmundito
Copy link
Contributor

What

Fixes #12443 - An issue where the confirmation modal would appear while adding a new transformation while creating a connection.

The confirmation modal now only appears when editing the transformation but not when saving. Additionally, it ensures that the "Save connection" button is disabled while creating or editing a transformation:

Screen Shot 2022-06-02 at 10 52 21
Screen Shot 2022-06-02 at 10 51 28

How

When adding or editing a transformation, it generates a form within the main connection form. Previously, it was tracking the context of the parent form. Now it's using its own context so that the dirtiness of the form is tracked separately.

Tests

Manually tested:

  1. Add a transformation when creating a new connection
  2. Edit transformation within an existing connection
  3. Edit transformation while creating a new connection
  4. Cancel edit transformation and save a connection
  5. Navigate away while editing or adding new transformations during creating or editing a connection
  6. Verify that submit button is disabled while creating or editing a transformation in both create connection and edit transformations pages

@edmundito edmundito requested a review from a team as a code owner June 2, 2022 14:54
@github-actions github-actions bot added area/platform issues related to the platform area/frontend Related to the Airbyte webapp labels Jun 2, 2022
@edmundito edmundito force-pushed the edmundito/fix-create-edit-transformation branch from 7981f47 to 493f29e Compare June 3, 2022 14:27
… a transformation

* Add properties to track start/end editing in TransformationField
* Bubble up prop tracking to ConnectionForm
* Update CreateControls submit button to disable when form is not valid or a transformation is editing
…n transformation page

* Update FormCard to be able to disable submit button and disable if form is not valid
* Update TransformationView to track if transformation is edited to enable and disable submit button
@edmundito edmundito force-pushed the edmundito/fix-create-edit-transformation branch from 493f29e to ba38b0a Compare June 3, 2022 19:14
Copy link
Contributor

@teallarson teallarson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally as described.

@edmundito edmundito merged commit ab4028d into master Jun 10, 2022
@edmundito edmundito deleted the edmundito/fix-create-edit-transformation branch June 10, 2022 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unsaved Changes modal appearing in error on Custom Transformation form
2 participants