Start persisting call and optin events #761
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR enables persistence of call and optin events by uncommenting them in the persistence configuration and adding temporary time-gating logic for backfill purposes.
- Enables persistence for call created/received events and optin started/stopped events
- Adds temporary time gate to only persist these events after August 25, 2025
- Reorders the eventPersistence map entries alphabetically
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| // time gate events types we need to backfill | ||
| // TODO remove | ||
| if e.Type() == events.TypeCallCreated || e.Type() == events.TypeCallReceived || e.Type() == events.TypeOptInStarted || e.Type() == events.TypeOptInStopped { | ||
| return e.CreatedOn().After(time.Date(2025, 8, 25, 20, 0, 0, 0, time.UTC)) |
There was a problem hiding this comment.
The hardcoded date in the time gate should be extracted to a constant or configuration to improve maintainability and make it easier to update when the TODO is addressed.
| return e.CreatedOn().After(time.Date(2025, 8, 25, 20, 0, 0, 0, time.UTC)) | |
| return e.CreatedOn().After(timeGateDate) |
Copilot uses AI. Check for mistakes.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #761 +/- ##
==========================================
- Coverage 51.02% 51.02% -0.01%
==========================================
Files 250 250
Lines 15100 15105 +5
==========================================
+ Hits 7705 7707 +2
- Misses 6588 6590 +2
- Partials 807 808 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.