Skip to content

Conversation

@satoshiotomakan
Copy link
Collaborator

This pull request removes support for the "managed coin register" functionality from the Aptos transaction builder and related code. The main changes include deleting the code for creating and handling managed coin registration transactions, as well as removing the associated Protobuf message and test logic.

Removed managed coin registration functionality:

  • Deleted the managed_coin_register function from aptos_move_packages.rs, which built the transaction payload for registering a managed coin.
  • Removed the register_token branch and method from TransactionFactory in transaction_builder.rs, eliminating support for building these transactions. [1] [2]
  • Removed the import of managed_coin_register from transaction_builder.rs.

Protobuf and test cleanup:

  • Deleted the ManagedTokensRegisterMessage message and its usage from Aptos.proto, and removed the corresponding field from the SigningInput message. [1] [2]
  • Removed test logic for the register_token operation from signer.rs.

@github-actions
Copy link

github-actions bot commented Sep 9, 2025

Binary size comparison

➡️ aarch64-apple-ios:

- 14.07 MB
+ 14.10 MB 	 +26 KB

➡️ aarch64-apple-ios-sim:

- 14.07 MB
+ 14.10 MB 	 +28 KB

➡️ aarch64-linux-android:

- 18.56 MB
+ 18.56 MB 	 -9 KB

➡️ armv7-linux-androideabi:

- 15.53 MB
+ 15.52 MB 	 -6 KB

➡️ wasm32-unknown-emscripten:

- 13.23 MB
+ 13.22 MB 	 -4 KB

@satoshiotomakan satoshiotomakan merged commit 6e515ac into master Sep 10, 2025
15 checks passed
@satoshiotomakan satoshiotomakan deleted the chore/aptos-cleanup branch September 10, 2025 10:01
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.

4 participants