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

Transaction Status: Add support for token-group and token-metadata interface instructions #980

Merged
merged 7 commits into from
Apr 24, 2024

Conversation

buffalojoec
Copy link

Problem

The transaction-status crate, used by the RPC to parse instructions with
jsonParsed encoding, supports most of the Token-2022 extensions. However, it
doesn't support the extensions that implement interfaces - such as
TokenMetadata and TokenGroup.

Developers would benefit from being able to view parsed instructions - such as
initializeTokenMetadata and updateGroupMaxSize - in RPC jsonParsed
instructions.

Summary of Changes

Add support for the Token Group and Token Metadata interface within
parse_token to enable parsing of interface instructions.

@buffalojoec buffalojoec force-pushed the token-interface-rpc-parsing branch from 3696991 to c842f7e Compare April 22, 2024 23:05
@buffalojoec buffalojoec requested a review from joncinque April 22, 2024 23:05
@buffalojoec buffalojoec marked this pull request as ready for review April 22, 2024 23:05
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 88.99676% with 68 lines in your changes are missing coverage. Please review.

Project coverage is 81.9%. Comparing base (b745dc9) to head (c842f7e).
Report is 7 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master     #980    +/-   ##
========================================
  Coverage    81.9%    81.9%            
========================================
  Files         853      855     +2     
  Lines      231955   232251   +296     
========================================
+ Hits       189984   190384   +400     
+ Misses      41971    41867   -104     

Copy link

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Thanks so much for taking care of this! Just a few suggestions, but this is very close

transaction-status/src/parse_token.rs Show resolved Hide resolved
transaction-status/Cargo.toml Outdated Show resolved Hide resolved
Copy link

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@buffalojoec buffalojoec added the automerge automerge Merge this Pull Request automatically once CI passes label Apr 23, 2024
Copy link

mergify bot commented Apr 24, 2024

automerge label removed due to a CI failure

@mergify mergify bot removed the automerge automerge Merge this Pull Request automatically once CI passes label Apr 24, 2024
@buffalojoec buffalojoec force-pushed the token-interface-rpc-parsing branch from 652a621 to 32d3e7a Compare April 24, 2024 00:57
@buffalojoec buffalojoec added the automerge automerge Merge this Pull Request automatically once CI passes label Apr 24, 2024
@mergify mergify bot merged commit 38c1946 into anza-xyz:master Apr 24, 2024
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants