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

πŸ’„ (smpl) [DSDK-518]: Update device selection according to figma #356

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

jdabbech-ledger
Copy link
Contributor

@jdabbech-ledger jdabbech-ledger commented Oct 4, 2024

πŸ“ Description

Select Device in a drawer
Update e2e tests according to the new UI

❓ Context

βœ… Checklist

Pull Requests must pass the CI and be code reviewed. Set as Draft if the PR is not ready.

  • Covered by automatic tests.
  • Changeset is provided
  • Impact of the changes:
device-selection.mp4

🧐 Checklist for the PR Reviewers

  • The code aligns with the requirements described in the linked JIRA or GitHub issue.
  • The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
  • There are no undocumented trade-offs, technical debt, or maintainability issues.
  • The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
  • Any new dependencies have been justified and documented.

Copy link

vercel bot commented Oct 4, 2024

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

Name Status Preview Comments Updated (UTC)
device-sdk-ts-sample ❌ Failed (Inspect) Oct 22, 2024 3:32pm

@jdabbech-ledger jdabbech-ledger changed the base branch from develop to feat/dsdk-440-ble-device-connection October 4, 2024 11:57
Copy link
Contributor

github-actions bot commented Oct 4, 2024

Messages
βœ… Danger: All checks passed successfully! πŸŽ‰

Generated by 🚫 dangerJS against ef58ea5

@jdabbech-ledger jdabbech-ledger changed the title [SDK] [TS] Update device selection according to figma πŸ’„ (smpl) [DSDK-518]: Update device selection according to figma Oct 4, 2024
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-518-device-selection-design branch from 08d982f to bf37266 Compare October 4, 2024 13:10
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-440-ble-device-connection branch from cee6a89 to 2305693 Compare October 4, 2024 13:43
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-518-device-selection-design branch from bf37266 to aeb3700 Compare October 4, 2024 13:44
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-440-ble-device-connection branch 2 times, most recently from 2b2f555 to 0ceb52d Compare October 4, 2024 14:00
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-518-device-selection-design branch from aeb3700 to ed8d956 Compare October 4, 2024 14:01
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-440-ble-device-connection branch from 0ceb52d to 6faac6a Compare October 4, 2024 14:39
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-518-device-selection-design branch from ed8d956 to 1233680 Compare October 4, 2024 14:41
@jiyuzhuang jiyuzhuang requested a review from a team October 4, 2024 14:42
@@ -3,7 +3,7 @@ import { Flex, Text, Drawer } from "@ledgerhq/react-ui";

export const StyledDrawer: React.FC<{
title: string;
description: string;
description?: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

[COULD] move optional parameter at the end

@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-440-ble-device-connection branch 3 times, most recently from a3e09a1 to 2fb8880 Compare October 14, 2024 09:04
@jdabbech-ledger jdabbech-ledger force-pushed the feat/dsdk-440-ble-device-connection branch 8 times, most recently from ccbda27 to dd61a4e Compare October 18, 2024 12:14
Base automatically changed from feat/dsdk-440-ble-device-connection to develop October 18, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hodl Do not merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants