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

LLD FAT-421 - Custom lock screen full flow #1352

Merged
merged 39 commits into from
Oct 27, 2022

Conversation

ofreyssinet-ledger
Copy link
Contributor

@ofreyssinet-ledger ofreyssinet-ledger commented Sep 22, 2022

πŸ“ Description

Details in the associated Jira tickets.

screenshots test report: (I'm linking this here bc it's getting drowned in the nb of msgs on this PR)
#1352 (comment)

❓ Context

  • Impacted projects: ledger-live-desktop ledger-live-mobile live-common
  • Linked resource(s): FAT-421

βœ… Checklist

  • Test coverage
  • Atomic delivery
  • No breaking changes

πŸ“Έ Demo

πŸš€ Expectations to reach

Please make sure you follow these Important Steps.

Pull Requests must pass the CI and be internally validated in order to be merged.

@vercel
Copy link

vercel bot commented Sep 22, 2022

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Updated
live-common-tools ❌ Failed (Inspect) Oct 27, 2022 at 3:22PM (UTC)
3 Ignored Deployments
Name Status Preview Updated
ledger-live-github-bot ⬜️ Ignored (Inspect) Oct 27, 2022 at 3:22PM (UTC)
native-ui-storybook ⬜️ Ignored (Inspect) Oct 27, 2022 at 3:22PM (UTC)
react-ui-storybook ⬜️ Ignored (Inspect) Oct 27, 2022 at 3:22PM (UTC)

@changeset-bot
Copy link

changeset-bot bot commented Sep 22, 2022

πŸ¦‹ Changeset detected

Latest commit: 8ee7f55

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@ledgerhq/react-ui Patch
live-mobile Patch
@ledgerhq/live-common Patch
ledger-live-desktop Patch
@ledgerhq/live-cli Patch
live-common-tools Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added common Has changes in live-common desktop Has changes in LLD mobile Has changes in LLM translations Translation files have been touched labels Sep 22, 2022
@@ -2,12 +2,12 @@ import "@ledgerhq/react-ui/assets/fonts";
import React, { useMemo } from "react";
Copy link
Contributor Author

@ofreyssinet-ledger ofreyssinet-ledger Sep 22, 2022

Choose a reason for hiding this comment

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

in this file I did 3 things:

  • renaming of slightly confusing names
  • cleanup of useless logic (the const v3SelectedPalettes = ... and palettesAny: any = palettes)
  • adding v2Palettes to colors because in one case (ContrastChoice component) we need to access a v2 background color in a v3 component which therefore cannot access that color (because style provider context etc.)

@codecov
Copy link

codecov bot commented Sep 22, 2022

Codecov Report

Base: 0.00% // Head: 47.94% // Increases project coverage by +47.94% πŸŽ‰

Coverage data is based on head (e2fecdb) compared to base (6de53ce).
Patch has no changes to coverable lines.

❗ Current head e2fecdb differs from pull request most recent head 8ee7f55. Consider uploading reports for the commit 8ee7f55 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #1352       +/-   ##
============================================
+ Coverage         0   47.94%   +47.94%     
============================================
  Files            0      691      +691     
  Lines            0    30445    +30445     
  Branches         0     7977     +7977     
============================================
+ Hits             0    14597    +14597     
- Misses           0    14636    +14636     
- Partials         0     1212     +1212     
Flag Coverage Ξ”
test 47.94% <ΓΈ> (?)

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

Impacted Files Coverage Ξ”
...kages/hw-app-eth/src/services/ledger/loadConfig.ts 100.00% <0.00%> (ΓΈ)
...bs/ledger-live-common/src/families/solana/logic.ts 24.61% <0.00%> (ΓΈ)
libs/ledger-live-common/src/apps/react.ts 85.00% <0.00%> (ΓΈ)
...ages/hw-app-eth/src/modules/EIP712/EIP712.utils.ts 93.75% <0.00%> (ΓΈ)
...ive-common/src/families/ethereum/modules/erc721.ts 26.22% <0.00%> (ΓΈ)
...-live-common/src/families/ethereum/modules/send.ts 22.47% <0.00%> (ΓΈ)
...live-common/src/families/algorand/hw-getAddress.ts 50.00% <0.00%> (ΓΈ)
...ilies/solana/api/chain/instruction/system/index.ts 36.36% <0.00%> (ΓΈ)
libs/ledgerjs/packages/hw-app-trx/src/utils.ts 61.40% <0.00%> (ΓΈ)
...-live-common/src/families/stellar/hw-getAddress.ts 50.00% <0.00%> (ΓΈ)
... and 681 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

β˜” View full report at Codecov.
πŸ“’ Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link

github-actions bot commented Sep 22, 2022

@ofreyssinet-ledger

Screenshots: ❌

It seems this PR contains screenshots that are different from the base branch.
If you are sure all those changes are correct, you can comment on this PR with /generate-screenshots to update those screenshots.

Make sure all the changes are correct before running the command, as it will commit and push the new result to the PR.

linux

Actual Diff Expected
custom-image-adjust-4-rotate-once-actual custom-image-adjust-4-rotate-once-diff custom-image-adjust-4-rotate-once-expected
custom-image-adjust-4-rotate-once-actual custom-image-adjust-4-rotate-once-diff custom-image-adjust-4-rotate-once-expected
v3-onboarding-complete-actual v3-onboarding-complete-diff v3-onboarding-complete-expected
v3-onboarding-complete-actual v3-onboarding-complete-diff v3-onboarding-complete-expected
v3-onboarding-complete-actual v3-onboarding-complete-diff v3-onboarding-complete-expected
v3-onboarding-complete-actual v3-onboarding-complete-diff v3-onboarding-complete-expected

windows

Actual Diff Expected
discover-actual discover-diff discover-expected
discover-actual discover-diff discover-expected

@ofreyssinet-ledger
Copy link
Contributor Author

/generate-screenshots

@ofreyssinet-ledger
Copy link
Contributor Author

/generate-screenshots

@github-actions
Copy link

github-actions bot commented Oct 20, 2022

Copy link
Contributor

@alexandremgo alexandremgo left a comment

Choose a reason for hiding this comment

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

πŸ”₯

@ofreyssinet-ledger ofreyssinet-ledger merged commit 0d7e0f7 into develop Oct 27, 2022
@ofreyssinet-ledger ofreyssinet-ledger deleted the feat/FAT-231-lld-customimage branch October 27, 2022 16:22
@ofreyssinet-ledger ofreyssinet-ledger changed the title LLD FAT-421 LLD FAT-421 - Custom lock screen full flow Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
common Has changes in live-common desktop Has changes in LLD mobile Has changes in LLM screenshots Screenshots have been updated translations Translation files have been touched ui Has changes in the design system library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants