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
[LLD] - [LIVE-12264] - Add Wallet Sync activation screen #6782
base: develop
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ 5 Ignored Deployments
|
29e5925
to
21a9128
Compare
apps/ledger-live-desktop/src/renderer/screens/settings/sections/General/WalletSyncRow.tsx
Outdated
Show resolved
Hide resolved
apps/ledger-live-desktop/src/renderer/screens/settings/sections/General/WalletSyncRow.tsx
Outdated
Show resolved
Hide resolved
apps/ledger-live-desktop/src/renderer/screens/settings/sections/General/WalletSyncRow.tsx
Outdated
Show resolved
Hide resolved
apps/ledger-live-desktop/src/renderer/screens/settings/sections/General/WalletSyncRow.tsx
Outdated
Show resolved
Hide resolved
apps/ledger-live-desktop/src/renderer/screens/settings/sections/General/WalletSyncRow.tsx
Outdated
Show resolved
Hide resolved
On the ticket |
More generally, I think that we should start to write our Integration tests now for Wallet Sync. |
21a9128
to
443e92d
Compare
|
||
const WalletSyncRow = () => { | ||
const lldWalletSync = useFeature("lldWalletSync"); | ||
|
||
const walletSync = useSelector(walletSyncSelector); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can create a selector for activated
only, in order to not "export" all the state
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Felt like it wasn't necessary right now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you use the same file naming conventions for this folder, please?
PascalCase + index.tsx
WalletSync/index.tsx
"cta": "Create a backup" | ||
}, | ||
"alreadySync": { | ||
"title": "Already created a back-up on another Device ?", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"title": "Already created a back-up on another Device ?", | |
"title": "Already created a back-up on another Device?", |
<LogoWrapper> | ||
<Icons.Refresh size="L" color={colors.constant.purple} /> | ||
</LogoWrapper> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This icon should not have 70% opacity.
285fe06
to
9343252
Compare
90d89f5
to
1c4baf8
Compare
1c4baf8
to
7930a6b
Compare
7930a6b
to
027ef52
Compare
β Checklist
npx changeset
was attached.π Description
This PR adds the Wallet Sync activation screen on LLD.
Note: It is missing shadows on the icons.
β Context
π§ Checklist for the PR Reviewers