Connection error when changing permissions makes Scatter think it succeeded #38
Description
I have worked with a user in China today who was using a VPN (otherwise can't use Telegram) and trying to update permissions on an account to a Ledger key.
After going to Change Permissions and selecting the new (Ledger) key in the dropdown and trying to confirm the user ends up at an empty screen with an infinite loading spinner:
In the console we can confirm a connection error:
The most confusing part is that afterwards Scatter seemed to think that the permissions update was successful! Internally it showed the (previously unused) ledger key being associated to the account. In a block explorer we confirmed that no keys were updated.
To clean up we used the "Unlink Unselected Accounts" option (by the way was a bit confusing for me to negate it like that, isn't "Unlink Selected Accounts" more intuitive, and then on the next screen I would explicitly select all the ones I want to unlink?), and after using "Refresh Accounts" again it did indeed correctly no longer have that key associated with any account.