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

feat: retroactively handle strategy #45

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

0xnigir1
Copy link
Collaborator

🤖 Linear

Closes GIT-206 GIT-207 GIT-187

Description

We want to retroactively process events for strategies that were previously unhandleable and now we have the corresponding handler implemented. For this we create the RetroactiveProcessor an independent class from the main Orchestrator.
Also, we introduce a new table so we checkpoint every event we process and can safely manage a critical error and restart retroactive processing without processing an event twice.

Checklist before requesting a review

  • I have conducted a self-review of my code.
  • I have conducted a QA.
  • If it is a core feature, I have included comprehensive tests.

@0xnigir1 0xnigir1 self-assigned this Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant