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

Add basic v2 transactions and blocks #109

Merged
merged 5 commits into from
Oct 15, 2024
Merged

Add basic v2 transactions and blocks #109

merged 5 commits into from
Oct 15, 2024

Conversation

chris124567
Copy link
Member

@chris124567 chris124567 commented Oct 4, 2024

  • Update schema for v2 transactions
  • Store v2 transaction arbitrary data
  • Add V2Transactions function for retrieving v2 transactions
  • Update Block to fill out V2BlockData if applicable
  • Add endpoints to retrieve v2 transactions and chain indices

Branch is based on add-api-test, will change to master when that is merged.

@chris124567 chris124567 changed the title Add v2 Add basic v2 transactions and blocks Oct 4, 2024
Base automatically changed from add-api-test to master October 7, 2024 20:38
@chris124567 chris124567 marked this pull request as ready for review October 12, 2024 00:18
@chris124567 chris124567 requested review from ChrisSchinnerl and n8maninger and removed request for ChrisSchinnerl October 12, 2024 00:26
@@ -80,6 +84,13 @@ type EventTransaction struct {
Fee types.Currency `json:"fee"`
}

// An EventV2Transaction represents a v2 transaction that affects the wallet.
type EventV2Transaction struct {
Copy link
Member

@n8maninger n8maninger Oct 15, 2024

Choose a reason for hiding this comment

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

Eventually these need to move to the coreutils types

@n8maninger n8maninger merged commit 4d34627 into master Oct 15, 2024
8 checks passed
@n8maninger n8maninger deleted the add-v2 branch October 15, 2024 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants