Testing connection - clearing old alert box when testing connection

[utkarsharma2]

When testing the connection for the second time the alert box with the old value is confusing, it can mean two things

1. If the error is still the same
2. The connection call is not returned yet

Better to clear it before making a testing call.

[pankajkoti]

Nice, it confuses me too, I often go to the browser console to check the response of the newer requests with modified payloads 😄.
Not much expertise with JS here, but if hiding the alert box is the right solution, I like the idea and the fix! Thank you for this! 👏🏽

[utkarsharma2]

@pankajkoti This was the easiest solution. Ideally, there should be a loader/spinner next to the test button, but that in comparison is more work.

[bbovenzi]

lgtm

I agree. We should probably also add a loading indicator to the test connection button. But that can be another PR.