-
Notifications
You must be signed in to change notification settings - Fork 38
feat:adding instruction parsing for native programs #98
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
base: master
Are you sure you want to change the base?
feat:adding instruction parsing for native programs #98
Conversation
|
@Nagaprasadvr currently added System program , Token program , Assosiated Token Program program , Memo program |
|
i think this can go inside FetchTransaction and we ask the user whether to parse ixs or not looks good can u attach some pictures showing parsing , try to get a tx which covers most of the ixs also maybe a sep one for memo |
So how should the flow look like Yes ill attach the screenshot |
it should be after asking for signature step |
Okey as an optional feature |
|
@Nagaprasadvr here are the screenshots Transfer with compute budget setting |
|
Maybe we can display it in a different way so it's easy to understand the whole tx flow , ig we can ditch the table and use something else for this let me think about this |
Yeah feels a bit cluttered |
|
Hey @Vinayapr23 what we could do as well is to get known programs name and put a mask on them. Like if program_id starts with "Tokenkeg..." we display Token Program instead of the address... wdyt @Nagaprasadvr ? |
Hey @BretasArthur1 I think iam already doing that I believe |
|
hey @Vinayapr23 a tree like structure showing ixs , cpi , accounts etc use ptree rust lib just build this tree structure and then we can iterate on showing data specific to each ix |
Okey Ill utilise this and rework |
|
nice |
Will work on this |
0cdda90 to
fd0c57e
Compare
fd0c57e to
5960e3e
Compare
|
@Nagaprasadvr changed to specified format using ptree
|
Nagaprasadvr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks clean , few comments and gtg
|
can u please attach how does a custom ix look like ( maybe a swap ix ) |
|
how do we handle CPI ? |
Ill check this and address the rest of the comments and let you know |
u can look at explorer and get a swap tx signature |
|
@Vinayapr23 let me know when u reolve these comments or u need any help |
Sure ser ill resolve it soon |
@Nagaprasadvr Not handling CPI now only top level instructions |











Description
Implements instruction parsing for Solana native programs. This PR adds a new "Parse Instructions" command to the Transaction module that decodes and displays transaction instructions in a human-readable format.
Related Issue
Partially closes #44
Fixes #
Type of Change
Before Submitting
masterbranchCode Quality
make all-checksand all checks pass