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.

Sign up for GitHub

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

Jump to bottom

feat: Add events to the transaction model in GraphQL #43

Merged
merged 8 commits into from
Jun 10, 2024
Merged

feat: Add events to the transaction model in GraphQL #43

merged 8 commits into from
Jun 10, 2024

Conversation

jinoosss
Copy link
Member

@jinoosss jinoosss commented May 3, 2024

Descriptions

Based on issue #134,
Add Events to the GraphQL transaction model.
The Event structure follows emit_event.go.

This is what works for this PR.

  • Add Events to the transaction response model.
  • Adds the feature to filter transactions by events.

Examples

Filter by transaction event and get the event

image

@jinoosss jinoosss marked this pull request as ready for review May 7, 2024 11:37
@jinoosss jinoosss requested a review from a team as a code owner May 7, 2024 11:37
@adr-sk adr-sk mentioned this pull request May 8, 2024
Open
@jinoosss jinoosss requested a review from ajnavarro May 8, 2024 06:58
@jinoosss jinoosss self-assigned this May 9, 2024
ajnavarro
ajnavarro requested changes May 9, 2024
View reviewed changes
serve/graph/model/transaction.go Outdated Show resolved Hide resolved
serve/graph/model/transaction.go Outdated Show resolved Hide resolved
ajnavarro
ajnavarro requested changes May 9, 2024
View reviewed changes
serve/graph/model/transaction.go Outdated Show resolved Hide resolved
@Kouteki Kouteki mentioned this pull request May 16, 2024
Open
@michelleellen michelleellen mentioned this pull request May 22, 2024
Open
@jinoosss jinoosss mentioned this pull request Jun 5, 2024
Closed
@zivkovicmilos zivkovicmilos merged commit faa744b into main Jun 10, 2024
3 checks passed
@zivkovicmilos zivkovicmilos deleted the feat/add-event-in-transaction branch June 10, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajnavarro ajnavarro ajnavarro approved these changes

@zivkovicmilos zivkovicmilos Awaiting requested review from zivkovicmilos zivkovicmilos is a code owner

Assignees

@jinoosss jinoosss

Labels
None yet
Projects
🤝🏻 Partner: Onbloc
Status: Done
🧙‍♂️gno.land core team
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jinoosss @ajnavarro @zivkovicmilos