-
Notifications
You must be signed in to change notification settings - Fork 302
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
feat(llm): new add account drawer on assets screen with WS entrypoint #7064
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 5 Ignored Deployments
|
d58c323
to
525a6e1
Compare
525a6e1
to
f9aa552
Compare
@@ -0,0 +1,93 @@ | |||
/* eslint-disable @typescript-eslint/no-unused-vars */ |
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.
If we follow the guidelines we can't add a drawers
folder here.
We should put AddAccount (PascalCase) under screens
or components
folder.
https://ledgerhq.atlassian.net/wiki/spaces/WXP/pages/4508058220/Code+Guidelines+-+React+Native#Folder-organization
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.
LGTM
f459232
to
3a4d6c5
Compare
3a4d6c5
to
1c2414b
Compare
896736a
to
e76560a
Compare
e76560a
to
4938d10
Compare
6a6ae89
to
0e1d53a
Compare
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.
🚀
✅ Checklist
npx changeset
was attached.📝 Description
Create a new drawer component on add account. This new drawer has new UI and an entry point to walletSync. This entrypoint is hidden under a feature flag. Also update translations related to WS.
Some adjustments have been added to make integration tests work
TODO
:❓ Context
🧐 Checklist for the PR Reviewers