0xStation / proposer

1 stars 0 forks source link

check for multisig address when fetching account email #794

Closed kristencheung closed 1 year ago

kristencheung commented 1 year ago

What's included in this pr + description

https://linear.app/station/issue/WEB-1059/i-added-and-verified-the-email-but-doesnt-show-up-in-the-email-field

Mind tried to verify the station express email but it wasn't showing up. The reason for that is because we have authentication to check if the wallet we're fetching the email for is the same as the connected wallet. This logic doesn't work for multi-sigs though.

I added a new optional arg for chain Id to authenticate a signer of a multisig for the getAccountEmail query.

Tested fetching the email on staging

image
linear[bot] commented 1 year ago
WEB-1059 I added and verified the email but doesn't show up in the email field

[Screen Shot 2022-11-29 at 12.01.20 PM.png](https://uploads.linear.app/e186bf5f-a55d-4799-9233-adcf54b8efd4/08fafdf1-7763-4ce5-bb6e-6d57d118606b/45453afd-f21c-405b-8099-7384f202d0a1)

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
web ✅ Ready (Inspect) Visit Preview Nov 30, 2022 at 11:01PM (UTC)