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

Token Transfer Processor - Derive classic events from operation and operation results #5609

Merged
merged 87 commits into from
Mar 21, 2025
Merged

Token Transfer Processor - Derive classic events from operation and operation results #5609

merged 87 commits into from
Mar 21, 2025

Conversation

karthikiyer56
Copy link
Contributor

@karthikiyer56 karthikiyer56 commented Feb 19, 2025 •
edited
Loading

This PR deals with deriving TokenTransferEvents from operation and operation results as described in #5580
This PR is strictly dealing with classic and classic operations, i.e the PR implements the ask from #5592

@karthikiyer56 karthikiyer56 linked an issue Feb 19, 2025 that may be closed by this pull request
Generate TokenTransferEvents from operation and operation results for classic operations #5592
Closed
@karthikiyer56 karthikiyer56 force-pushed the karthik/ttp-classic-operations branch from 159c84a to 36946d4 Compare February 20, 2025 05:08
@karthikiyer56 karthikiyer56 force-pushed the karthik/ttp-classic-operations branch from 36946d4 to 571679c Compare February 20, 2025 23:41
@socket-security Socket Security
Copy link

socket-security bot commented Mar 21, 2025 •
edited
Loading

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
golang/github.com/google/go-cmp@v0.6.0 ➜ v0.7.0 eval 0 470 kB

View full report↗︎

tamirms
tamirms approved these changes Mar 21, 2025
View reviewed changes
Copy link
Contributor

@tamirms tamirms left a comment

Choose a reason for hiding this comment

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

🎉 🎉 🎉

@karthikiyer56 karthikiyer56 merged commit 37d384a into master Mar 21, 2025
23 checks passed
@sreuland sreuland mentioned this pull request Mar 21, 2025
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chowbao chowbao chowbao left review comments

@sydneynotthecity sydneynotthecity sydneynotthecity left review comments

@tamirms tamirms tamirms approved these changes

@Shaptic Shaptic Awaiting requested review from Shaptic

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Generate TokenTransferEvents from operation and operation results for classic operations
4 participants
@karthikiyer56 @tamirms @chowbao @sydneynotthecity