Remove the Test and Save buttons after a successful publish and replace with a 'View Details` button that takes the user to the details
Added some error handling for super edge cases where the publish might have a call failed during it.
Updating the Test and Save & Publish buttons to show Testing and Saving & Publishing while there is an action.
Added Saved to the active state so forms cannot be changed after being saved.
misc
Update Icon Noir. This was initially part of an experiment but the work is valid so leaving it in
Adding in a disabled state for remove for when it is removing
Added a check that the call to the server to remove the binding actually worked before removing it from the store
Made the removal of bindings a bit more robust to potential errors. This really only happened if a user would click on the remove button of multiple bindings quickly.
Fixed some random DOM issues that Chrome called out where ids and names were missing on inputs/buttons.
Fixed issue where time travel was editable during an async action to the server
Tests
Manually tested
scenarios you manually tested
Automated tests
unit testing covered
Screenshots
If applicable - please include some screenshots of the new UI
Issues
https://github.com/estuary/ui/issues/913
Changes
913
Test
andSave & Publish
buttons to showTesting
andSaving & Publishing
while there is an action.Saved
to the active state so forms cannot be changed after being saved.misc
id
s andname
s were missing on inputs/buttons.Tests
Manually tested
Automated tests
Screenshots
If applicable - please include some screenshots of the new UI