-
Notifications
You must be signed in to change notification settings - Fork 324
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
Feature/llm card #7774
Feature/llm card #7774
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 5 Skipped Deployments
|
apps/ledger-live-mobile/src/components/RootNavigator/CardLiveAppNavigator.tsx
Outdated
Show resolved
Hide resolved
); | ||
}; | ||
|
||
export default function CardAppNavigator(_props?: Record<string, unknown>) { |
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.
same as above
Also please update the type to use an actual type instead of Record<string, unknown>
apps/ledger-live-mobile/src/components/RootNavigator/CardLiveAppNavigator.tsx
Outdated
Show resolved
Hide resolved
..._props.route, | ||
params: { | ||
goToURL: _props.route.params?.goToURL, | ||
lastScreen: _props.route.params?.lastScreen, | ||
platform: _props.route.params?.platform || defaultPlatform, |
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 piece is a little unclear to me, why are we spreading the route object but then assigning onto the params
values from the object we just spread?
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.
Refactor this part of the code, ensuring that we only fill in parameters that have not been set.
✅ Checklist
npx changeset
was attached.📝 Description
Implement Ledger CarD Web in LLM
❓ Context
https://ledgerhq.atlassian.net/browse/LIVE-13970
https://ledgerhq.atlassian.net/browse/LIVE-13741
🧐 Checklist for the PR Reviewers