Skip to content

Conversation

@satoshiotomakan
Copy link
Collaborator

@satoshiotomakan satoshiotomakan commented Dec 8, 2023

Description

Move BNB Beacon chain to Rust.

How to test

Run Rust, C++, Android and iOS tests

Breaking Changes

  • Remove the previously deprecated TWTransactionCompilerBuildInput FFI.

TODO

  • Implement all order types
  • Implement Compiler functionality
  • Remove C++ tests

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.

@satoshiotomakan satoshiotomakan marked this pull request as ready for review December 11, 2023 17:28
Copy link
Contributor

@lamafab lamafab left a comment

Choose a reason for hiding this comment

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

LGTM. Those /// cbindgen:ignore comments are expected (for the codegen I assume), right?

@satoshiotomakan
Copy link
Collaborator Author

Unfortunately, yes

LGTM. Those /// cbindgen:ignore comments are expected (for the codegen I assume), right?

@satoshiotomakan satoshiotomakan merged commit 2a63cda into master Dec 13, 2023
@satoshiotomakan satoshiotomakan deleted the s/rust-bnb branch December 13, 2023 16:22
zkrypt-crossbar pushed a commit to Cramiumlabs/wallet-core that referenced this pull request Jun 15, 2025
* [BNB]: Generate BNB Beacon chain skeleton files in Rust

* [BNB]: Implement `BinanceAddress`, add Address tests

* [BNB]: Add TBinance address tests

* [BNB]: Add Transaction and its encoding, prehashing

* Add Amino encoding

* [BNB]: Sign a transaction

* TODO Add tests, handle all message types

* [BNB]: Add a signing test

* [BNB]: Add `SendOrder`, `CancelOrder`

* [BNB]: Add Token orders

* [BNB]: Add `TokenBurnOrder`, `HTLTOrder`

* [BNB]: Add `DepositHTLT`

* [BNB]: Add `ClaimHTLTOrder`, `RefundHTLTOrder`

* [BNB]: Add `TransferOutOrder`

* [BNB]: Add `SideChainDelegate`, `SideChainRedelegate`, `SideChainUndelegate` orders

* [BNB]: Add `TimeLock`, `TimeRelock`, `TimeUnlock` orders

* [BNB]: Implement Transaction Compiler

* [BNB]: Increase code coverage

* [BNB]: Replace C++ implementation

* [BNB]: Move TX preimage implementation to `JsonPreimager`

* Remove C++ Signing tests

* [BNB]: Extend `CoinEntry` documentation

* [CI] Trigger CI

* [BNB]: Remove `TransactionCompilerBuildInput`

* [BNB]: Avoid duplicating code

* [BNB]: Add fuzz tests
zkrypt-crossbar pushed a commit to Cramiumlabs/wallet-core that referenced this pull request Jun 21, 2025
* [BNB]: Generate BNB Beacon chain skeleton files in Rust

* [BNB]: Implement `BinanceAddress`, add Address tests

* [BNB]: Add TBinance address tests

* [BNB]: Add Transaction and its encoding, prehashing

* Add Amino encoding

* [BNB]: Sign a transaction

* TODO Add tests, handle all message types

* [BNB]: Add a signing test

* [BNB]: Add `SendOrder`, `CancelOrder`

* [BNB]: Add Token orders

* [BNB]: Add `TokenBurnOrder`, `HTLTOrder`

* [BNB]: Add `DepositHTLT`

* [BNB]: Add `ClaimHTLTOrder`, `RefundHTLTOrder`

* [BNB]: Add `TransferOutOrder`

* [BNB]: Add `SideChainDelegate`, `SideChainRedelegate`, `SideChainUndelegate` orders

* [BNB]: Add `TimeLock`, `TimeRelock`, `TimeUnlock` orders

* [BNB]: Implement Transaction Compiler

* [BNB]: Increase code coverage

* [BNB]: Replace C++ implementation

* [BNB]: Move TX preimage implementation to `JsonPreimager`

* Remove C++ Signing tests

* [BNB]: Extend `CoinEntry` documentation

* [CI] Trigger CI

* [BNB]: Remove `TransactionCompilerBuildInput`

* [BNB]: Avoid duplicating code

* [BNB]: Add fuzz tests
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