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

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

Conversation

matter-q
Copy link
Contributor

@matter-q matter-q commented Feb 1, 2023

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

@matter-q matter-q 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 🥶

@matter-q matter-q 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
@matter-q matter-q 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
@matter-q matter-q requested a review from edmundito February 6, 2023 20:52
@matter-q matter-q merged commit e8386a2 into master Feb 6, 2023
@matter-q matter-q 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