Migrate account

[tanmoyAtb]

closes #829

[render]

Your Render PR Server URL is https://chainsafex-dev-pr-834.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c1cv9l5ua9vq4a1gnnu0.

[render]

Your Render PR Server URL is https://files-ui-dev-pr-834.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c1cv9ldua9vq4a1gno3g.

[render]

Your Render PR Server URL is https://files-landing-dev-pr-834.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c1cv9ltua9vq4a1gnomg.

[tanmoyAtb]

Some instances where i'm confused and need clearing up

• logout from encryption password screen should clear threshould key

• submitting encryption password does not clear shouldInitializeAccount

• refreshing the page after reaching migrate screen does not set shouldInitializeAccount to true on reload.

• sidenote, on login with legacy account, MissingShares screen flashes for a bit

[FSM1]

• logout from encryption password screen should clear threshould key

Yeah that is correct. This will reset the tkey instance to a fresh one. There is a log out method exposed on tkey context that then cascades the logout to all the other layers

• submitting encryption password does not clear shouldInitializeAccount

That needs to be rectified. Can probably call the setter just before returning from that submit function (inside the tkey context)

• refreshing the page after reaching migrate screen does not set shouldInitializeAccount to true on reload.

This can be solved by adding another conditional for when the Migrate Account shoud be shown (isMasterPasswordSet && !secured)

• sidenote, on login with legacy account, MissingShares screen flashes for a bit

Will need to do some digging. I actually need to create some accounts and test this flow out.

[FSM1]

Left some comments. Looking good overall

[FSM1]

Added some minor fixes

[FSM1]

Great work @tanmoyAtb 🎉

[Tbaut]

Did a first fly-by, found some nits, I will test it now :)

[Tbaut]

I'm for creating an issue rather than adding this here, which we might ignore.

[tanmoyAtb]

Definitely better to have an issue

[Tbaut]

added it #849

[Tbaut]

It works well!
I found 2/3 more nits:

• the images are not aligned with the text, I show it for the first 2.
• there's a space missing somehow after the link
• I feel this modal should be a little bigger, so that it breathes a bit more (the mock design has it bigger)