Skip to content

Fix displaying transaction on network#513

Merged
popenta merged 1 commit intomainfrom
fix-tx-on-network-printing
Apr 9, 2025
Merged

Fix displaying transaction on network#513
popenta merged 1 commit intomainfrom
fix-tx-on-network-printing

Conversation

@popenta
Copy link
Copy Markdown
Collaborator

@popenta popenta commented Apr 9, 2025

After being processed, the transaction is displayed as is fetched from the network.

@popenta popenta self-assigned this Apr 9, 2025
Comment thread pyproject.toml
[project]
name = "multiversx-sdk-cli"
version = "10.1.0"
version = "10.1.1"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Debatable - maybe minor version, since it's a theoretical breaking change in the output format.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's have it as patch, as it is 👍


if self.transaction_on_network:
transaction_on_network_dict = transaction_on_network_to_dictionary(self.transaction_on_network)
transaction_on_network_dict = self.transaction_on_network.raw
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good, now we return the data as it comes from the API (no change - except of the field omissions below) 👍

@popenta popenta merged commit 8ac2d4e into main Apr 9, 2025
12 checks passed
@popenta popenta deleted the fix-tx-on-network-printing branch April 9, 2025 09:18
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.

3 participants