Details
-
Bug
-
Resolution: Done
-
Major
-
None
-
None
-
None
Description
Context
DataSourceDialog has a button for testing a PostgreSQL DB connection. This button sends a request to the provided DB endpoint and shows an error/success message appropriately. However neither the button or its functionality disabled when clicking, so that the request can be sent many times before any response arrives.
What
The "Test" button has to be disabled when clicking it and as long as the network request is running. Once a response arrives (and the error/success message is shown) the button will be re-enabled. Also while the request is running, a loading spinner should be displayed.
Why
This is a clear UX anti-pattern.
How
This exact functionality is already implemented in HookFormGroup and it should be replicated.
Extras
Whenever the DB host, port, Database, Username or Password change, the test result should be reset (any message and icons gone).