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

Ton integration #7270

Closed
wants to merge 65 commits into from
Closed

Ton integration #7270

wants to merge 65 commits into from

Conversation

hedi-edelbloute
Copy link
Member

✅ Checklist

  • npx changeset was attached.
  • Covered by automatic tests.
  • Impact of the changes: coin-ton
    • ...

📝 Description

Ton integration (original pr #6983)

❓ 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)

@hedi-edelbloute hedi-edelbloute requested review from a team as code owners July 5, 2024 07:31
Copy link

vercel bot commented Jul 5, 2024

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

5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
ledger-live-docs ⬜️ Ignored (Inspect) Visit Preview Jul 22, 2024 2:51pm
ledger-live-github-bot ⬜️ Ignored (Inspect) Visit Preview Jul 22, 2024 2:51pm
native-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Jul 22, 2024 2:51pm
react-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Jul 22, 2024 2:51pm
web-tools ⬜️ Ignored (Inspect) Visit Preview Jul 22, 2024 2:51pm

@live-github-bot live-github-bot bot added desktop Has changes in LLD mobile Has changes in LLM common Has changes in live-common ui Has changes in the design system library ledgerjs Has changes in the ledgerjs open source libs tools Has changes in tools translations Translation files have been touched cli labels Jul 5, 2024
@live-github-bot
Copy link
Contributor

live-github-bot bot commented Jul 5, 2024

Mobile Bundle Checks

Comparing 28296ae against 281e8c7.

⚠️ main.ios.jsbundle bundle size significantly increased: 59.5mb -> 59.9mb. Please check if this is expected.
⚠️ main.android.jsbundle bundle size significantly increased: 59.5mb -> 59.9mb. Please check if this is expected.

Desktop Bundle Checks

Comparing 8e9543a against 8553b3e.

⚠️ renderer bundle size significantly increased: 39.9mb -> 40.3mb. Please check if this is expected.
⚠️ debug library is now duplicated in main (regression)
⚠️ zod library is now duplicated in renderer (regression)

Copy link

[Bot] Testing with 'Nitrogen' 💰 1 miss funds ($0.00) ⏲ 2806ms

💰 1 specs may miss funds: TON

What is the bot and how does it work? Everything is documented here!

⚠️ 1 spec hints
  • Spec TON:
    • There are not enough accounts (1) to cover all mutations (2).
      Please increase the account target to at least 3 accounts
Details of the 0 mutations

Spec TON (failed)

Spec TON found 1 TON accounts. Will use TON 2.1.0 on nanoS 2.1.0
undefined: 0 TON (0ops) (UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0 on 44'/607'/0'/0'/0'/0') ton#0 js:2:ton:3f0a4cc222f4d9b4d8bcfb5d0cb4a8908bbcb4a5e375c7ef745e6e78b780669d:ton

This SEED does not have TON. Please send funds to UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0

Details of the 2 uncovered mutations

Spec TON (2)

  • Send ~50%:
  • Transfer Max:
Portfolio ($0.00) – Details of the 1 currencies
Spec (accounts) State Remaining Runs (est) funds?
TON (1) 0 ops , 0 TON ($0.00) UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0
undefined: 0 TON (0ops) (UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0 on 44'/607'/0'/0'/0'/0') ton#0 js:2:ton:3f0a4cc222f4d9b4d8bcfb5d0cb4a8908bbcb4a5e375c7ef745e6e78b780669d:ton
Performance ⏲ 2806ms

Time spent for each spec: (total across mutations)

Spec (accounts) preload scan re-sync tx status sign op broadcast test destination test
TOTAL 1.12ms 949ms N/A N/A N/A N/A N/A N/A
TON (0) 1.12ms 949ms N/A N/A N/A N/A N/A N/A

What is the bot and how does it work? Everything is documented here!

Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] None 0 23.4 kB steve.kite

View full report↗︎

Copy link

[Bot] Testing with 'Nitrogen' ✅ 2 txs ($2.00) ⏲ 88.1s

✅ 1 specs are successful: TON

What is the bot and how does it work? Everything is documented here!

⚠️ 1 spec hints
  • Spec TON:
    • There are not enough accounts (2) to cover all mutations (2).
      Please increase the account target to at least 3 accounts
Details of the 2 mutations

Spec TON (2)

Spec TON found 2 TON accounts. Will use TON 2.1.0 on nanoS 2.1.0
undefined: 0.3 TON (1ops) (UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0 on 44'/607'/0'/0'/0'/0') ton#0 js:2:ton:3f0a4cc222f4d9b4d8bcfb5d0cb4a8908bbcb4a5e375c7ef745e6e78b780669d:ton
undefined: 0 TON (0ops) (UQCgFq3YfNA6MbXU__1AX1iSOy-iS5O5iIvqVprRQINAfACO on 44'/607'/0'/0'/1'/0') ton#1 js:2:ton:160444af87a5a38726dbe60ed81906095cb3e196ea1f7d5ec1be614945346359:ton
necessary accounts resynced in 0.21ms
▬ TON 2.1.0 on nanoS 2.1.0
→ FROM undefined: 0.3 TON (1ops) (UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0 on 44'/607'/0'/0'/0'/0') ton#0 js:2:ton:3f0a4cc222f4d9b4d8bcfb5d0cb4a8908bbcb4a5e375c7ef745e6e78b780669d:ton
max spendable ~0.3
★ using mutation 'Transfer Max'
→ TO undefined: 0 TON (0ops) (UQCgFq3YfNA6MbXU__1AX1iSOy-iS5O5iIvqVprRQINAfACO on 44'/607'/0'/0'/1'/0') ton#1 js:2:ton:160444af87a5a38726dbe60ed81906095cb3e196ea1f7d5ec1be614945346359:ton
✔️ transaction 
SEND MAX
TO UQCgFq3YfNA6MbXU__1AX1iSOy-iS5O5iIvqVprRQINAfACO
STATUS (1389ms)
  amount: 0.294610398 TON
  estimated fees: 0.005389602 TON
  total spent: 0.3 TON
errors: 
warnings: 
✔️ has been signed! (2512ms) 
✔️ broadcasted! (205ms) optimistic operation: 
  -0.3 TON           OUT        7cQQQWcMMsogJzkGePA5hOkuEaaNuKc1I2V0yvGwS6A= 2024-07-22T15:14
✔️ operation confirmed (31.2s): 
  -0.299733331 TON   OUT        IZHNifmd2t7TKPBg47Nkrpl7u9aFojIaFE+2ItoYgcQ= 2024-07-22T15:14
✔️ undefined: 0 TON (2ops) (UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0 on 44'/607'/0'/0'/0'/0') ton#0 js:2:ton:3f0a4cc222f4d9b4d8bcfb5d0cb4a8908bbcb4a5e375c7ef745e6e78b780669d:ton (! sum of ops 0.000266669 TON)(in 31.2s)
✔️ destination operation 
  +0.294610397 TON   IN         IZHNifmd2t7TKPBg47Nkrpl7u9aFojIaFE+2ItoYgcQ= 2024-07-22T15:14
(in 10.9s)

necessary accounts resynced in 0.21ms
▬ TON 2.1.0 on nanoS 2.1.0
→ FROM undefined: 0.29461 TON (1ops) (UQCgFq3YfNA6MbXU__1AX1iSOy-iS5O5iIvqVprRQINAfACO on 44'/607'/0'/0'/1'/0') ton#1 js:2:ton:160444af87a5a38726dbe60ed81906095cb3e196ea1f7d5ec1be614945346359:ton
max spendable ~0.29461
★ using mutation 'Send ~50%'
→ TO undefined: 0 TON (2ops) (UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0 on 44'/607'/0'/0'/0'/0') ton#0 js:2:ton:3f0a4cc222f4d9b4d8bcfb5d0cb4a8908bbcb4a5e375c7ef745e6e78b780669d:ton
✔️ transaction 
SEND  0.147305199 TON
TO UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0
STATUS (2119ms)
  amount: 0.147305199 TON
  estimated fees: 0.005402401 TON
  total spent: 0.1527076 TON
errors: 
warnings: 
✔️ has been signed! (2397ms) 
✔️ broadcasted! (196ms) optimistic operation: 
  -0.1527076 TON     OUT        VG6v2ck62C1IO/U2kspqKlW3jdsOJah3sAPHCBS6Y3U= 2024-07-22T15:15
✔️ operation confirmed (21.4s): 
  -0.152440931 TON   OUT        ulVNv0NqGW04QLCpu12JbinaoV2MEVTmCQ+o2OUruoo= 2024-07-22T15:15
✔️ undefined: 0.141902 TON (2ops) (UQCgFq3YfNA6MbXU__1AX1iSOy-iS5O5iIvqVprRQINAfACO on 44'/607'/0'/0'/1'/0') ton#1 js:2:ton:160444af87a5a38726dbe60ed81906095cb3e196ea1f7d5ec1be614945346359:ton (! sum of ops 0.142169466 TON)(in 21.4s)
✔️ destination operation 
  +0 TON             NONE       ulVNv0NqGW04QLCpu12JbinaoV2MEVTmCQ+o2OUruoo= 2024-07-22T15:15
(in 11.2s)


Portfolio ($2.00) – Details of the 1 currencies
Spec (accounts) State Remaining Runs (est) funds?
TON (2) 6 ops (+5), 0.3 TON ($2.00) 👍 73 UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0
undefined: 0.146995 TON (4ops) (UQDL7vAIogYGacmfO0xTS1bwPEMJNh1g1Jliwq2p-qWkS1J0 on 44'/607'/0'/0'/0'/0') ton#0 js:2:ton:3f0a4cc222f4d9b4d8bcfb5d0cb4a8908bbcb4a5e375c7ef745e6e78b780669d:ton
undefined: 0.141902 TON (2ops) (UQCgFq3YfNA6MbXU__1AX1iSOy-iS5O5iIvqVprRQINAfACO on 44'/607'/0'/0'/1'/0') ton#1 js:2:ton:160444af87a5a38726dbe60ed81906095cb3e196ea1f7d5ec1be614945346359:ton
Performance ⏲ 88.1s

Time spent for each spec: (total across mutations)

Spec (accounts) preload scan re-sync tx status sign op broadcast test destination test
TOTAL 1.11ms 1993ms 0.42ms 3.5s 4.9s 401ms 52.6s 22.1s
TON (2) 1.11ms 1993ms 0.42ms 3.5s 4.9s 401ms 52.6s 22.1s

What is the bot and how does it work? Everything is documented here!

@emmanuelm41
Copy link
Contributor

@hedi-edelbloute should we close this PR?

Copy link

There as been no activity on this PR for the last 14 days. Please consider closing this PR.

@github-actions github-actions bot added the Stale label Aug 14, 2024
@github-actions github-actions bot closed this Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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 Stale tools Has changes in tools translations Translation files have been touched ui Has changes in the design system library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants