Feat/add ton jettons #26

emmanuelm41 · 2024-04-30T14:26:02Z

✅ Checklist

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

📝 Description

Replace this text by a clear and concise description of what this pull request is about and why it is needed. Be sure to explain the problem you're addressing and the solution you're proposing.
For libraries, you can add a code sample of how to use it.
For bugfixes, you can explain the previous behavior and how it was fixed.
In case of visual features, please attach screenshots or video recordings to demonstrate the changes.

❓ Context

JIRA or GitHub link:

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

emmanuelm41

Pretty solid so far @ayelenmurano!!! It looks great!!

libs/coin-modules/coin-ton/src/cli-transaction.ts

libs/coin-modules/coin-ton/src/js-estimateMaxSpendable.ts

libs/coin-modules/coin-ton/src/js-prepareTransaction.ts

libs/coin-modules/coin-ton/src/js-signOperation.ts

libs/coin-modules/coin-ton/src/utils.ts

libs/ledgerjs/packages/cryptoassets/src/data/jetton.json

apps/ledger-live-desktop/cryptoassets.md

lawRathod · 2024-05-23T15:45:41Z

tools/actions/turbo-affected/build/main.js

@@ -18595,8 +18595,10 @@ var package_default = {
    "coin:bitcoin": "pnpm --filter coin-bitcoin",
    "coin:evm": "pnpm --filter coin-evm",
    "coin:framework": "pnpm --filter coin-framework",
+    "coin:tester": "pnpm --filter coin-tester",


This might be unrelated change...

You are right. I'll delete it. Thanks for realizing it!

lawRathod · 2024-05-23T15:49:46Z

libs/env/src/env.ts

@@ -775,8 +775,9 @@ const envDefinitions = {
    parser: intParser,
    desc: "Configure the low battery percentage threshold",
  },
+  // TODO: TON switch to mainnet
  API_TON_ENDPOINT: {


We could move this to config.ts like in near https://github.com/LedgerHQ/ledger-live/blob/e7ed028716bccb9dc29aa2fc672ecc7a3e78276a/libs/coin-modules/coin-near/src/config.ts

I think they recently added this way of adding env config variables.

Really good idea! I hadn't seen it. I'll add the endpoint and key from there.

lawRathod

All looks pretty great to me!! 🚀

emmanuelm41

Looks to me too 🚀

Upgrade pnpm to version 9.4.0

This reverts commit 1843b41.

feat(lld): refactor of breadcrumb for nft gallery

* fix: LIVE-13279 * feat: stop caching results from fallback + throws correctly when network error * chore: changeset * chore: var renaming * chore: const in caps

…Q#7229) chore: add scenario that fails on the trustchain app

* fix: coin-tester polkadot runtime upgrade * fix: update changelog * fix: update polkadot nano app firmware version in coin-tester * fix: lint error

* fix: revert ignoring all swaps error * chore: changeset

[QAA][Playwright] Fix playwright mocked test for features flags prod

…os_firebase_testing [QAA-137][Speculos] Change back firebase env to testing

Upgrade Turborepo to 2.0.6

…rbo-2" This reverts commit ef86807, reversing changes made to c34461a.

Revert Turborepo 2

… the first one

emmanuelm41 commented Apr 30, 2024

View reviewed changes

ayelenmurano force-pushed the feat/ton branch from dff0968 to 1b56c0a Compare May 7, 2024 18:10

ayelenmurano force-pushed the feat/add-ton-jettons branch from 11f60a9 to 102dbc5 Compare May 7, 2024 19:08

lawRathod reviewed May 23, 2024

View reviewed changes

lawRathod approved these changes May 23, 2024

View reviewed changes

emmanuelm41 commented May 27, 2024

View reviewed changes

ayelenmurano force-pushed the feat/add-ton-jettons branch 2 times, most recently from b4a06ec to 7a1ed78 Compare May 30, 2024 15:22

ayelenmurano force-pushed the feat/add-ton-jettons branch 4 times, most recently from af132dc to 3c3a8c5 Compare July 4, 2024 17:42

KVNLS and others added 17 commits July 5, 2024 15:20

fix: Upgrade pnpm to version 9.4.0

88c8e57

fix: update actions build

779c0f5

Merge pull request LedgerHQ#7248 from LedgerHQ/fix/upgrade-pnpm

1cc909e

Upgrade pnpm to version 9.4.0

Revert "ci: change testing firebase env to prod"

02d5b95

This reverts commit 1843b41.

💥feat(lld): nft breadcrumb (LedgerHQ#7249)

5f4e68d

feat(lld): refactor of breadcrumb for nft gallery

fix: LIVE-13279 (LedgerHQ#7256)

6353cea

* fix: LIVE-13279 * feat: stop caching results from fallback + throws correctly when network error * chore: changeset * chore: var renaming * chore: const in caps

chore(trustchain): add twoAddMembersFollowedByDeviceAdd test (LedgerH…

a67a2d1

…Q#7229) chore: add scenario that fails on the trustchain app

Support LIVE-13070 coin tester polkadot runtime upgrade (LedgerHQ#7181)

f0ed580

* fix: coin-tester polkadot runtime upgrade * fix: update changelog * fix: update polkadot nano app firmware version in coin-tester * fix: lint error

fix: Upgrade Turborepo to 2.0.6

1dc438e

fix: address pr request changes

a6f1db1

fix: revert ignoring all swaps error (LedgerHQ#7277)

289996a

* fix: revert ignoring all swaps error * chore: changeset

Merge pull request LedgerHQ#7251 from LedgerHQ/support/playwright_test

d491591

[QAA][Playwright] Fix playwright mocked test for features flags prod

ci: change back firebase env to testing for speculos

e36582f

Merge pull request LedgerHQ#7281 from LedgerHQ/support/qaa-137_specul…

c34461a

…os_firebase_testing [QAA-137][Speculos] Change back firebase env to testing

Merge pull request LedgerHQ#7276 from LedgerHQ/fix/upgrade-turbo-2

ef86807

Upgrade Turborepo to 2.0.6

Revert "Merge pull request LedgerHQ#7276 from LedgerHQ/fix/upgrade-tu…

00fe9a8

…rbo-2" This reverts commit ef86807, reversing changes made to c34461a.

Merge pull request LedgerHQ#7285 from LedgerHQ/fix/revert-turborepo-2

5305d5e

Revert Turborepo 2

ayelenmurano and others added 26 commits July 22, 2024 12:03

feat(ton): add tests and run lint and typecheck commands

6060dba

feat(ton): adjust jetton transfer fees, refactor get transaction status

e6d3f50

feat(ton): add jetton icons

08e91ea

feat(ton): add custom config for ton

f28d91d

add changeset

7016bc7

refactor(ton): rebase adjustments

e83cd27

feat(ton): Adjust test changes

11c8a25

feat(ton): add usdt as jetton

b305751

refactor(ton): rebase adjustments

f15deb6

refactor(ton): solve comments

5da03b1

refactor(ton): code style adjustments

50dbf9e

refactor(ton): solve comments

443ed1d

feat(ton): Add NOT as jetton and add MEM icon

073822f

refactor(ton): add fees payed in the last step, move comment entry to…

3d06c04

… the first one

fix(ton): add a missing package and fix an import

c117a43

Add unit tests

f081688

refactor(ton): solve comments and adjust mobile version

717788c

refactor(ton): rebase adjustments and adjust jetton decimals

c913abc

refactor(ton): Delete a condition

372c10f

refactor(ton): run lint:fix command

02e5dcd

feat(ton): adjust endpoints to ledger reverse proxy

08aca46

feat(ton): delete jetton integration

4987814

feat(ton): snapshot update

5efe391

feat(ton): add fees as suboperation on IN txs and update snapshot

7141649

feat(ton): downgrade expo-random and add deleted icons

d3f1df0

feat(ton): solve mobile issue on android

5fc15d5

emmanuelm41 force-pushed the feat/add-ton-jettons branch from 01d342f to 5fc15d5 Compare July 22, 2024 15:10

fix: rebase issues

3445486

emmanuelm41 force-pushed the feat/add-ton-jettons branch from 96a344b to 3445486 Compare July 22, 2024 16:57

fix: pnpm lock and podfile.lock

958bf92

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/add ton jettons #26

Feat/add ton jettons #26

emmanuelm41 commented Apr 30, 2024

emmanuelm41 left a comment

lawRathod May 23, 2024

ayelenmurano May 24, 2024

lawRathod May 23, 2024

ayelenmurano May 24, 2024

lawRathod left a comment

emmanuelm41 left a comment

Feat/add ton jettons #26

Are you sure you want to change the base?

Feat/add ton jettons #26

Conversation

emmanuelm41 commented Apr 30, 2024

✅ Checklist

📝 Description

❓ Context

🧐 Checklist for the PR Reviewers

emmanuelm41 left a comment

Choose a reason for hiding this comment

lawRathod May 23, 2024

Choose a reason for hiding this comment

ayelenmurano May 24, 2024

Choose a reason for hiding this comment

lawRathod May 23, 2024

Choose a reason for hiding this comment

ayelenmurano May 24, 2024

Choose a reason for hiding this comment

lawRathod left a comment

Choose a reason for hiding this comment

emmanuelm41 left a comment

Choose a reason for hiding this comment