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.

Sign up for GitHub

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

Jump to bottom

feat: solana tokens support [LIVE-12069] #8119

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

feat: solana tokens support [LIVE-12069] #8119

wants to merge 28 commits into from

Conversation

Justkant
Copy link
Contributor

@Justkant Justkant commented Oct 16, 2024 •
edited
Loading

✅ Checklist

  • npx changeset was attached.
  • Covered by automatic tests.
  • Impact of the changes:
    • Solana accounts in LLD, LLM. Send tokens, receive tokens, view tokens

📝 Description

We need to fix and merge the solana priority fees before this one

Original work: hoodieshq#2

📝 Description

  • Finalized integration with SPL tokens on Solana.

Details:

  • Updated sub accounts sychronisation [coin-solana solana]
  • Added BURN operation type [LLD, libs/coin-framework, libs/ledgerjs/packages/types-live]
  • Added account sub header for displaying alert message for frozen tokens [LLD, LLM]
  • Disable staking modal for solana tokens [LLD]
  • Added additinal warning messages [LLM, LLD]
  • Fixed tx fee of token account (it should be in SOL) [LLM]
  • Implement functions (assignFromTokenAccountRaw, assignToTokenAccountRaw) for serialization of TokenAccount custom fields (like token "frozen" state) [libs/coin-framework, libs/ledgerjs/packages/types-live]
  • Enabled existing tokens integration tests and added additional few in bridge.integration.test.ts [coin-solana solana]
  • Added speculos tests for tokens [coin-solana solana]
  • Updated bridge.integration.test.ts snapshot [LLC solana]

Screenshots

image
image
image
image
image image image image

❓ Context

🧐 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.
  • Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

@Justkant Justkant self-assigned this Oct 16, 2024
@Justkant Justkant requested review from a team as code owners October 16, 2024 18:02
@vercel Vercel
Copy link

vercel bot commented Oct 16, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web-tools ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 18, 2024 3:11pm
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
ledger-live-docs ⬜️ Ignored (Inspect) Visit Preview Dec 18, 2024 3:11pm
ledger-live-github-bot ⬜️ Ignored (Inspect) Visit Preview Dec 18, 2024 3:11pm
native-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Dec 18, 2024 3:11pm
react-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Dec 18, 2024 3:11pm

@live-github-bot live-github-bot bot added desktop Has changes in LLD mobile Has changes in LLM common Has changes in live-common ledgerjs Has changes in the ledgerjs open source libs translations Translation files have been touched labels Oct 16, 2024
@mikhd mikhd mentioned this pull request Oct 21, 2024
Open
9 tasks
@Justkant Justkant force-pushed the feat/solana-tokens-support branch from 7c21f2c to 9ddc7d7 Compare October 29, 2024 10:32
@Justkant Justkant force-pushed the feat/solana-tokens-support branch from 9ddc7d7 to 48f4d62 Compare November 4, 2024 08:18
@Justkant Justkant force-pushed the feat/solana-tokens-support branch from 48f4d62 to 18c1f45 Compare November 4, 2024 15:34
@Justkant Justkant force-pushed the feat/solana-tokens-support branch from 18c1f45 to daa5138 Compare November 13, 2024 10:33
@Justkant Justkant force-pushed the feat/solana-tokens-support branch from 075aba9 to 290295f Compare November 18, 2024 14:46
@live-github-bot live-github-bot bot added the cli label Nov 18, 2024
@Justkant Justkant requested a review from a team as a code owner November 18, 2024 16:12
Wozacosta
Wozacosta previously approved these changes Nov 19, 2024
View reviewed changes
Copy link
Contributor

@Wozacosta Wozacosta left a comment

Choose a reason for hiding this comment

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

🚀

@Justkant Justkant force-pushed the feat/solana-tokens-support branch from 6886149 to dc4e6a0 Compare November 19, 2024 16:31
@Justkant Justkant force-pushed the feat/solana-tokens-support branch from dc4e6a0 to 00943c9 Compare November 19, 2024 17:01
mikhd and others added 26 commits December 18, 2024 16:01
@mikhd @Justkant
feat: enable solana spl tokens import
0cd544c
@mikhd @Justkant
feat: finish solana tokens integration into bridge [coin-solana]
2bdb997
@mikhd @Justkant
chore: update solana bridge and speculos tests
1d4bc7f
@mikhd @Justkant
feat: display solana tokens in LLD
16731a5
@mikhd @Justkant
feat: display solana tokens in LLM
1784db9
@mikhd @Justkant
chore(LLC): add token account serialization test
714363f
@mikhd @Justkant
chore: add changeset
3d8a8ad
@mikhd @Justkant
fix(LLD, LLM): token summary and device screens
0dcf431
@Justkant
fix: non existing param used for SPL Tokens
359790b
@Justkant
fix: wrong chainId for spl tokens
4bb5372 
Used the import:cal-tokens to update only spl data
@Justkant
fix: lint issue on missing break
adefedd
@Justkant
chore: update bridge integration test snapshot for solana
45a8982
@Justkant
chore: update spl token snapshot
70b5b48
@Justkant
feat: add solana testnet and devnet in supported currencies list
a5dbb6c
@Justkant
fix: spl priority fees and transaction confirmation with blockHeight
6057308
@Justkant
chore: add support links
aad6cc2
@Justkant
test: fix with solana testnet and devnet addition
303f17f
@Justkant
chore: remove comments from review comments
3df39dd
@Justkant
fix: wallet-api spl usage
837f6d4
@Justkant
fix: wallet-api spl token support
d1ac359
@Justkant
test: fix test expectation
d87f79a
@Justkant
test: fix wrong findTokenById call and param
8ef7751
@Justkant
test: fix missing account type
fd707d6
@Justkant
feat: add trusted name resolution for signTransaction
22056fa
@Justkant
feat: update warning message for LNS
707b167
@Justkant
feat: clear sign SPL token address
ce62e15
@Justkant Justkant force-pushed the feat/solana-tokens-support branch from 24eef2f to ce62e15 Compare December 18, 2024 15:02
@Justkant Justkant requested a review from a team as a code owner December 18, 2024 15:02
@Justkant Justkant removed the Stale label Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wozacosta Wozacosta Awaiting requested review from Wozacosta

@themooneer themooneer Awaiting requested review from themooneer

At least 1 approving review is required to merge this pull request.

Assignees

@Justkant Justkant

Labels
cli common Has changes in live-common desktop Has changes in LLD ledgerjs Has changes in the ledgerjs open source libs mobile Has changes in LLM translations Translation files have been touched
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Justkant @Wozacosta @themooneer @mikhd