Skip to content

Allow users to cancel a currently running sync on a disabled connection #22241

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

Conversation

m19berger
Copy link
Contributor

What

Closes #22084

How

Changed logic to show/hide action buttons for the Connections Status Page

Now users have the possibility to cancel sync with a disabled connection

Loom

https://www.loom.com/share/bcb3f306ca6745ffbc2dab84fa570ffe

@m19berger m19berger requested a review from dizel852 February 1, 2023 21:02
@octavia-squidington-iii octavia-squidington-iii added the area/frontend Related to the Airbyte webapp label Feb 1, 2023
Copy link
Contributor

@dizel852 dizel852 left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Tested locally

Can be merged after Code Freeze 🥶

@m19berger m19berger requested a review from edmundito February 3, 2023 11:24
Copy link
Contributor

@edmundito edmundito left a comment

Choose a reason for hiding this comment

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

Tested and works great. I had some comments about the styling that can be improved.

- Changed logic to show/hide action buttons for Connections Status Page
@m19berger m19berger force-pushed the mark/allow-users-to-cancel_a-currently-running-sync-on-a-disabled-connection branch from 0e4d1b5 to 3812186 Compare February 6, 2023 19:17
- Changed logic to show/hide action buttons for Connections Status Page
@m19berger m19berger requested a review from edmundito February 6, 2023 20:52
@m19berger m19berger merged commit e8386a2 into master Feb 6, 2023
@m19berger m19berger deleted the mark/allow-users-to-cancel_a-currently-running-sync-on-a-disabled-connection branch February 6, 2023 22:13
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow users to cancel a currently running sync on a disabled connection
4 participants