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

Generate a parser for MASP transactions. #2732

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

murisi
Copy link
Contributor

@murisi murisi commented Feb 27, 2024

Describe your changes

Wrote a utility to convert the Borsh schemas for MASP Transactions and MaspBuilders into C parsers for Ledger hardware wallets. More specifically, the changes are as follows:

  • Now use a MASP branch with more correct Borsh schemas for Transaction and Builder
  • Fixed the BorshSchema for MaspBuilder
  • Implemented a C parser generator supporting a range of schema Definitions
  • Created an example named generate-parser to invoke the parser generator

Indicate on which release or other PRs this topic is based on

Namada v0.31.4
anoma/masp#71

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant