Skip to content

Commit

Permalink
Merge pull request #7291 from LedgerHQ/feat/dsdk-13125-live-assets
Browse files Browse the repository at this point in the history
💄 [LLD/LLM]: Europa assets
  • Loading branch information
jdabbech-ledger committed Jul 23, 2024
2 parents 0eb993f + 91c1e67 commit 033ebb8
Show file tree
Hide file tree
Showing 101 changed files with 1,038 additions and 354 deletions.
5 changes: 5 additions & 0 deletions .changeset/blue-fireants-change.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@ledgerhq/devices": patch
---

Rename Ledger Europa
5 changes: 5 additions & 0 deletions .changeset/old-vans-fry.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@ledgerhq/icons-ui": patch
---

Update Europa icon
5 changes: 5 additions & 0 deletions .changeset/strange-oranges-bake.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@ledgerhq/device-core": patch
---

shouldForceFirmwareUpdate: add <1.1.1 condition for europa
6 changes: 6 additions & 0 deletions .changeset/strong-bats-walk.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
"ledger-live-desktop": patch
"live-mobile": patch
---

Upadate Europa assets
18 changes: 17 additions & 1 deletion .github/workflows/build-desktop.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,10 +57,15 @@ jobs:
env:
NODE_OPTIONS: "--max-old-space-size=7168"
steps:
- name: generate token
id: generate-token
uses: tibdex/github-app-token@v1
with:
app_id: ${{ secrets.GH_BOT_APP_ID }}
private_key: ${{ secrets.GH_BOT_PRIVATE_KEY }}
- uses: actions/checkout@v4
with:
ref: ${{ inputs.ref || github.sha }}

- name: Setup git user
uses: LedgerHQ/ledger-live/tools/actions/composites/setup-git-user@develop

Expand Down Expand Up @@ -92,6 +97,17 @@ jobs:
uses: LedgerHQ/ledger-live/tools/actions/composites/setup-build-desktop@develop
with:
os: ${{ matrix.config.name }}
- name: Retrieving internal assets
uses: actions/checkout@v4
with:
ref: main
repository: LedgerHQ/ledger-live-internal-assets
token: ${{ steps.generate-token.outputs.token }}
path: ledger-live-internal-assets

- name: Replace assets
id: assets
run: pnpm desktop assets:replace

- name: Build the app
id: build-app
Expand Down
30 changes: 30 additions & 0 deletions .github/workflows/build-mobile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,12 @@ jobs:
env:
NODE_OPTIONS: "--max-old-space-size=7168"
steps:
- name: generate token
id: generate-token
uses: tibdex/github-app-token@v1
with:
app_id: ${{ secrets.GH_BOT_APP_ID }}
private_key: ${{ secrets.GH_BOT_PRIVATE_KEY }}
- uses: actions/checkout@v4
with:
ref: ${{ inputs.ref || github.sha }}
Expand Down Expand Up @@ -68,10 +74,19 @@ jobs:
- name: Get short SHA
id: slug
run: echo "sha8=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT
- name: Retrieving internal assets
uses: actions/checkout@v4
with:
ref: main
repository: LedgerHQ/ledger-live-internal-assets
token: ${{ steps.generate-token.outputs.token }}
path: ledger-live-internal-assets
- name: make local version
env:
VERSION: ${{ steps.version.outputs.clean }}-sha.${{ steps.slug.outputs.sha8 }}
run: cd apps/ledger-live-mobile && npm version $VERSION
- name: Replace assets
run: pnpm mobile assets:replace
- name: build the app
env:
ANDROID_KEYSTORE_PASS: staging
Expand All @@ -96,6 +111,12 @@ jobs:
env:
NODE_OPTIONS: "--max-old-space-size=7168"
steps:
- name: generate token
id: generate-token
uses: tibdex/github-app-token@v1
with:
app_id: ${{ secrets.GH_BOT_APP_ID }}
private_key: ${{ secrets.GH_BOT_PRIVATE_KEY }}
- uses: actions/checkout@v4
with:
ref: ${{ inputs.ref || github.sha }}
Expand All @@ -113,6 +134,15 @@ jobs:
turbo-server-token: ${{ secrets.TURBOREPO_SERVER_TOKEN }}
- name: install dependencies
run: pnpm i --filter="live-mobile..." --filter="ledger-live" --no-frozen-lockfile --unsafe-perm
- name: Retrieving internal assets
uses: actions/checkout@v4
with:
ref: main
repository: LedgerHQ/ledger-live-internal-assets
token: ${{ steps.generate-token.outputs.token }}
path: ledger-live-internal-assets
- name: Replace assets
run: pnpm mobile assets:replace
- name: bundle ios and android js
run: |
pnpm build:llm:deps --api="http://127.0.0.1:${{ steps.caches.outputs.port }}" --token="${{ secrets.TURBOREPO_SERVER_TOKEN }}" --team="foo"
Expand Down
3 changes: 2 additions & 1 deletion apps/ledger-live-desktop/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,8 @@
"test:playwright:clean": "git clean -fdX tests/artifacts",
"typecheck": "node scripts/typecheck.js",
"check": "pnpm lint; pnpm typecheck",
"unimported": "unimported"
"unimported": "unimported",
"assets:replace": "zx ./scripts/replace-assets.mjs"
},
"dependencies": {
"@braze/web-sdk": "^4.6.3",
Expand Down
14 changes: 14 additions & 0 deletions apps/ledger-live-desktop/scripts/replace-assets.mjs
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
#!/usr/bin/env zx
import "zx/globals";
import fs from "fs";
import path from "path";

const basePath = path.join(__dirname, "..", "..", "..");
const srcPath = path.join(__dirname, "..", "src");
const replaceAssetsPath = path.join(basePath, "ledger-live-internal-assets", "apps", "ledger-live-desktop", "src");

async function replaceAssets(){
await fs.promises.cp(replaceAssetsPath, srcPath, { recursive: true, force: true});
}

await replaceAssets();

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ const CustomImageDeviceAction: React.FC<Props> = withRemountableWrapper(props =>
deviceModelId={deviceModelId}
progress={progress}
source={source}
type={type}
/>
) : imageCommitRequested && device ? (
<RenderImageCommitRequested
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,11 +51,13 @@ export const RenderLoadingImage = ({
progress,
source,
deviceModelId,
type,
}: {
deviceModelId: CLSSupportedDeviceModelId;
device: Device;
progress: number | undefined;
source?: string;
type: Theme["theme"];
}) => {
const { t } = useTranslation();
return (
Expand All @@ -68,7 +70,7 @@ export const RenderLoadingImage = ({
>
<AnimationWrapper>
<FramedPicture
frameConfig={getFramedPictureConfig("transfer", deviceModelId)}
frameConfig={getFramedPictureConfig("transfer", deviceModelId, type)}
source={source}
loadingProgress={progress}
/>
Expand Down Expand Up @@ -114,7 +116,7 @@ export const RenderImageCommitRequested = ({
<DeviceBlocker />
<AnimationWrapper>
<FramedPicture
frameConfig={getFramedPictureConfig("transfer", deviceModelId)}
frameConfig={getFramedPictureConfig("preview", deviceModelId, type)}
source={source}
background={
<Animation animation={getDeviceAnimation(device.modelId, type, "confirmLockscreen")} />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ export type FramedPictureConfig = {
innerTop: number;
innerBottomHeight: number;
borderRightRadius: number;
borderLeftRadius?: number;
/** source of the background image */
backgroundSource?: string;
};
Expand Down Expand Up @@ -86,6 +87,7 @@ const FramedPicture: React.FC<Props> = ({
innerTop,
innerLeft,
borderRightRadius,
borderLeftRadius,
innerBottomHeight,
backgroundSource,
} = scaleFrameConfig(frameConfig, scale || 1);
Expand Down Expand Up @@ -129,6 +131,8 @@ const FramedPicture: React.FC<Props> = ({
objectFit: "fill",
borderTopRightRadius: borderRightRadius,
borderBottomRightRadius: borderRightRadius,
borderTopLeftRadius: borderLeftRadius,
borderBottomLeftRadius: borderLeftRadius,
}}
/>
) : null}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ const ImageGrayscalePreview: React.FC<Props> = props => {
/>
{previewResult ? (
<FramedPicture
frameConfig={getFramedPictureConfig("transfer", deviceModelId)}
frameConfig={getFramedPictureConfig("transfer", deviceModelId, theme.colors.palette.type)}
source={previewResult?.imageBase64DataUri}
/>
) : null}
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Original file line number Diff line number Diff line change
@@ -1,8 +1,11 @@
import { CLSSupportedDeviceModelId } from "@ledgerhq/live-common/device/use-cases/isCustomLockScreenSupported";
import staxTransferBackground from "./assets/transferBackground.png";
import staxTransferBackgroundLight from "./assets/staxTransferBackgroundLight.png";
import staxTransferBackgroundDark from "./assets/staxTransferBackgroundDark.png";
import europaTransferBackgroundLight from "./assets/europaTransferBackgroundLight.png";
import europaTransferBackgroundDark from "./assets/europaTransferBackgroundDark.png";
import { FramedPictureConfig } from "./FramedPicture";

export const staxTransferConfig: FramedPictureConfig = {
export const staxTransferConfig: Omit<FramedPictureConfig, "backgroundSource"> = {
frameHeight: 236,
frameWidth: 150,
innerHeight: 223,
Expand All @@ -12,10 +15,9 @@ export const staxTransferConfig: FramedPictureConfig = {
innerLeft: 1.5,
innerBottomHeight: 27,
borderRightRadius: 6.5,
backgroundSource: staxTransferBackground,
};

export const europaTransferConfig: FramedPictureConfig = {
export const europaTransferConfig: Omit<FramedPictureConfig, "backgroundSource"> = {
frameHeight: 236, // TODO:TBD
frameWidth: 150, // TODO:TBD
innerHeight: 223, // TODO:TBD
Expand All @@ -25,19 +27,68 @@ export const europaTransferConfig: FramedPictureConfig = {
innerLeft: 1.5, // TODO:TBD
innerBottomHeight: 27, // TODO:TBD
borderRightRadius: 6.5, // TODO:TBD
backgroundSource: staxTransferBackground, // TODO: replace by europaTransferBackground when it exists
};

const configs = {
type FrameConfigMap = {
[key in "transfer" | "preview"]: {
[modelId in CLSSupportedDeviceModelId]: {
light: FramedPictureConfig;
dark: FramedPictureConfig;
};
};
};

const configs: FrameConfigMap = {
transfer: {
stax: staxTransferConfig,
europa: europaTransferConfig,
stax: {
light: {
...staxTransferConfig,
backgroundSource: staxTransferBackgroundLight,
},
dark: {
...staxTransferConfig,
backgroundSource: staxTransferBackgroundDark,
},
},
europa: {
light: {
...europaTransferConfig,
backgroundSource: europaTransferBackgroundLight,
},
dark: {
...europaTransferConfig,
backgroundSource: europaTransferBackgroundDark,
},
},
},
preview: {
stax: {
light: {
...staxTransferConfig,
backgroundSource: staxTransferBackgroundLight,
},
dark: {
...staxTransferConfig,
backgroundSource: staxTransferBackgroundDark,
},
},
europa: {
light: {
...europaTransferConfig,
backgroundSource: europaTransferBackgroundLight,
},
dark: {
...europaTransferConfig,
backgroundSource: europaTransferBackgroundDark,
},
},
},
};

export function getFramedPictureConfig(
type: "transfer", // later on, there will be more types (like "preview" on LLM)
type: "transfer" | "preview", // later on, there will be more types (like "preview" on LLM)
deviceModelId: CLSSupportedDeviceModelId,
theme: "dark" | "light",
): FramedPictureConfig {
return configs[type][deviceModelId];
return configs[type][deviceModelId][theme];
}
Loading

0 comments on commit 033ebb8

Please sign in to comment.