Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restyle V0 extension migration UI #1790

Merged
merged 9 commits into from
Dec 6, 2023
Merged

Restyle V0 extension migration UI #1790

merged 9 commits into from
Dec 6, 2023

Conversation

lukaw3d
Copy link
Member

@lukaw3d lukaw3d commented Dec 5, 2023

Before After
Migrate-from-V0-extension-persisted-state-to-valid-RootState-1-chromium-linux Migrate-from-V0-extension-persisted-state-to-valid-RootState-1-chromium-linux
Migrate-from-V0-extension-persisted-state-to-valid-RootState-2-chromium-linux Migrate-from-V0-extension-persisted-state-to-valid-RootState-2-chromium-linux
My-Accounts-tab-should-get-a-private-key-1-chromium-linux My-Accounts-tab-should-get-a-private-key-1-chromium-linux

Mocks:

image

@lukaw3d lukaw3d requested a review from buberdds December 5, 2023 18:19
Copy link

github-actions bot commented Dec 5, 2023

Deployed to Cloudflare Pages

Latest commit: 325bcf69948eed6ddd1e22e5007b3dde80fd2000
Status:✅ Deploy successful!
Preview URL: https://5541b258.oasis-wallet.pages.dev

Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Merging #1790 (325bcf6) into master (0d36c15) will decrease coverage by 0.01%.
The diff coverage is 54.83%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1790      +/-   ##
==========================================
- Coverage   81.45%   81.44%   -0.01%     
==========================================
  Files         190      191       +1     
  Lines        5062     5067       +5     
  Branches      934      930       -4     
==========================================
+ Hits         4123     4127       +4     
- Misses        939      940       +1     
Flag Coverage Δ
cypress 45.89% <12.50%> (+0.08%) ⬆️
jest 77.33% <54.83%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/app/components/LanguageSelect/index.tsx 72.72% <ø> (ø)
src/app/components/Persist/LoginModalLayout.tsx 75.00% <100.00%> (ø)
src/app/components/RevealOverlayButton/index.tsx 50.00% <100.00%> (+5.55%) ⬆️
...lbar/Features/Account/ManageableAccountDetails.tsx 37.28% <100.00%> (+5.23%) ⬆️
...ponents/Toolbar/Features/NetworkSelector/index.tsx 86.66% <ø> (ø)
src/app/components/Persist/MigrateV0StateForm.tsx 32.30% <66.66%> (ø)
src/app/components/Sidebar/index.tsx 84.44% <33.33%> (-0.52%) ⬇️
src/app/components/PrivateKeyFormatter/index.tsx 47.61% <47.61%> (ø)

... and 2 files with indirect coverage changes

@lukaw3d lukaw3d changed the title Lw/restyle migration Restyle V0 extension migration UI Dec 5, 2023
@lukaw3d lukaw3d force-pushed the lw/restyle-migration branch 4 times, most recently from 56c2aff to e2e0ecc Compare December 5, 2023 22:27
/>
)}
{acknowledge && <PrivateKeyFormatter privateKey={wallet.privateKey!} />}
<RevealOverlayButton
Copy link
Contributor

@buberdds buberdds Dec 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That kinda sucks because priv key is in DOM already without acknowledge.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a little :/

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mocks even show the underlying element blurred, but I think blur might be reversible in screenshots

@buberdds
Copy link
Contributor

buberdds commented Dec 6, 2023

Should we consider moving Transak tests out of CI to some nightly build/tests in the near future?

@lukaw3d lukaw3d force-pushed the lw/restyle-migration branch from e2e0ecc to 325bcf6 Compare December 6, 2023 23:14
@lukaw3d
Copy link
Member Author

lukaw3d commented Dec 6, 2023

Should we consider moving Transak tests out of CI to some nightly build/tests in the near future?

Maybe yeah

@lukaw3d lukaw3d merged commit f11500c into master Dec 6, 2023
11 of 12 checks passed
@lukaw3d lukaw3d deleted the lw/restyle-migration branch December 6, 2023 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants