Skip to content

Conversation

@gupnik
Copy link
Collaborator

@gupnik gupnik commented Jun 6, 2025

Description

Fixes #4151

Breaking Change

This PR changes the amount type in TON API to accept big-endian bytes instead of uint64 to support large amount transfers.

How to test

Run tests across platforms

Types of changes

Bug fix (breaking change which fixes an issue)

Checklist

  • Create pull request as draft initially, unless its complete.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • If there is a related Issue, mention it in the description.

If you're adding a new blockchain

  • I have read the guidelines for adding a new blockchain.

@gupnik gupnik requested a review from satoshiotomakan as a code owner June 6, 2025 10:20
@github-actions
Copy link

github-actions bot commented Jun 6, 2025

Binary size comparison

➡️ aarch64-apple-ios:

- 14.01 MB
+ 14.02 MB 	 +4 KB

➡️ aarch64-apple-ios-sim:

- 14.01 MB
+ 14.02 MB 	 +4 KB

➡️ aarch64-linux-android:

- 18.48 MB
+ 18.49 MB 	 +5 KB

➡️ armv7-linux-androideabi:

- 15.46 MB
+ 15.47 MB 	 +4 KB

➡️ wasm32-unknown-emscripten:

- 13.16 MB
+ 13.17 MB 	 +4 KB

Copy link
Collaborator

@satoshiotomakan satoshiotomakan left a comment

Choose a reason for hiding this comment

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

LGTM, one minor change please before we merge the PR

@satoshiotomakan satoshiotomakan merged commit 9693d83 into master Jun 9, 2025
17 checks passed
@satoshiotomakan satoshiotomakan deleted the gupik/ton-fix branch June 9, 2025 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to send TON jetton transaction with amount more than Long max value

3 participants