Pubkey validation added to WatchAccountPopover

[sammanadh]

Mistakeny added a invalid public key in watchAccountPopover which caused the entire app to break. I guess when I added the invalid address to watch, it got stored and is causing this issue even after I restart the app. Next, I will clear stored address from memory, and figure our any measures we can take if such address somehow gets stored. But, for now, this PR should prevent user from entering the invalid address.

[sammanadh]

Ohh. It uses localstorage. For now the best place to validate public key seemes to the the pinAccount function. But I think implementing a redux middleware that makes sure the all the states that store addresses gets valid public key would be the best approach.

[nathanleclaire]

Looks great, thanks.

I think implementing a redux middleware that makes sure the all the states that store addresses gets valid public key would be the best approach.

Agreed, that might be a good follow up to work on if you're interested

One change around typing requested then LGTM

[nathanleclaire]

Almost there! Just one request to get rid of the !!validationError which was the idea behind making the type string | undefined

[nathanleclaire]

Oh nice, I can just edit your repo 🤯

LGTM