Skip to content

EventManager Improvements#133

Merged
itamarcps merged 2 commits intodevelopmentfrom
feature/event-manager-improv
Aug 16, 2024
Merged

EventManager Improvements#133
itamarcps merged 2 commits intodevelopmentfrom
feature/event-manager-improv

Conversation

@Pedrozo
Copy link
Contributor

@Pedrozo Pedrozo commented Aug 14, 2024

  • EventManager removed
  • Storage is now responsible for storing and retrieving events
  • eth_getLogs and eth_getTransactionReceipt no longer depends on State
  • No backward compatibility was broken. The events are still being saved in a different DB folder and using the same format

@Pedrozo Pedrozo requested a review from itamarcps August 14, 2024 16:35
@Pedrozo Pedrozo self-assigned this Aug 14, 2024
@Pedrozo Pedrozo added the enhancement New feature or request label Aug 14, 2024
@itamarcps itamarcps merged commit 3e243a9 into development Aug 16, 2024
@Jean-Lessa Jean-Lessa mentioned this pull request Aug 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Jean-Lessa Jean-Lessa Jean-Lessa approved these changes

@itamarcps itamarcps Awaiting requested review from itamarcps

Assignees

@Pedrozo Pedrozo

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Pedrozo @Jean-Lessa @itamarcps