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

fix(iota): fix Option<address> serialization in client_ptb #5143

Merged
merged 3 commits into from
Feb 4, 2025

Conversation

Thoralf-M
Copy link
Member

Description of change

Handle serialization for Option

in a PTB, since TypeTag for an Option is a vector<T> it needed to be handled extra

Links to any relevant issues

Fixes #5141

Type of change

  • Bug fix (a non-breaking change which fixes an issue)

How the change has been tested

cargo run client ptb \
--assign addr "some(@0xa5657935a0698bf654ce4ceb66b5a6d6a371a934a6afcfd1e191aa2ee460c8cc)" \
--move-call 0xf5097105292df26739f5d2ca10d5011e6bc54245ef7f1e0c708de3aa7f946544::example::optional_address_param addr \
--dry-run

Copy link

vercel bot commented Feb 2, 2025

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

4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
apps-backend ⬜️ Ignored (Inspect) Visit Preview Feb 3, 2025 9:00pm
apps-ui-kit ⬜️ Ignored (Inspect) Visit Preview Feb 3, 2025 9:00pm
rebased-explorer ⬜️ Ignored (Inspect) Visit Preview Feb 3, 2025 9:00pm
wallet-dashboard ⬜️ Ignored (Inspect) Visit Preview Feb 3, 2025 9:00pm

@iota-ci iota-ci added dev-tools Issues related to the Developer Tools Team sc-platform Issues related to the Smart Contract Platform group. labels Feb 2, 2025
@Thoralf-M Thoralf-M merged commit da1ba42 into develop Feb 4, 2025
41 checks passed
@Thoralf-M Thoralf-M deleted the dev-tools/option-address-serialization branch February 4, 2025 00:34
stefan-mysten pushed a commit to MystenLabs/sui that referenced this pull request Feb 11, 2025
# Description of change

Handle serialization for Option< address > in a PTB, since `TypeTag` for
an Option is a `vector<T>` it needed to be handled extra

Porting over iotaledger/iota#5143

## Type of change

- Bug fix (a non-breaking change which fixes an issue)

## How the change has been tested

Running this command connected to the testnet
```
cargo run client ptb \
--assign addr "some(@0x2d4b94541bbfcce2ed51e5b0fb73e02bd809e04f1a0601797a6028e6a00b6607)" \
--move-call 0xc9586aae0485f0975167ef0261db81a4011d3255cfa0128dc2f7cc3b72135aaf::example::optional_address_param addr \
--dry-run
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev-tools Issues related to the Developer Tools Team sc-platform Issues related to the Smart Contract Platform group.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to serialize Option<address> in CLI PTB
4 participants