Skip to content

feat: add TransactionRecordQuery #144

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

Dosik13
Copy link
Contributor

@Dosik13 Dosik13 commented Jun 19, 2025

Description

Implemented the TransactionRecordQuery class that retrieves transaction records from the network.

  • Add TransactionRecord class with transaction record data
  • Add unit tests for TransactionRecord class
  • Add TransactionRecordQuery class implementation for querying transaction records by transaction ID
  • Add _from_proto() method to TokenNftTransfer class for easier protobuf deserialization
  • Refactor TransactionReceipt to include transaction_id field so we can use it to execute TransactionRecordQuery
  • Add transaction record query example
  • Add integration/unit tests for TransactionRecordQuery
  • Update README.md in examples with transaction record query documentation
  • Update __init__.py file with TransactionRecord and TransactionRecordQuery class definitions

Related issue(s):

Fixes #138

Notes for reviewer:
Please, careful see the TransactionRecord and see if you like the way it is done.

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@Dosik13 Dosik13 marked this pull request as ready for review June 25, 2025 13:23
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.

Add TransactionRecordQuery
1 participant