Tested email analytics initializer - #29892
Conversation
…mailEventStorage` A few of these values were being sent into `init`, but not all the way "down" into `EmailAnalyticsServiceWrapper` and `NewsletterEmailEventStorage`.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run ghost:test:ci:integration |
✅ Succeeded | 2m 25s | View ↗ |
nx run ghost:test:integration |
✅ Succeeded | 3m 22s | View ↗ |
nx run ghost:test:e2e |
✅ Succeeded | 2m 50s | View ↗ |
nx run ghost:test:legacy |
✅ Succeeded | 2m 55s | View ↗ |
nx run-many -t test:unit -p ghost |
✅ Succeeded | 30s | View ↗ |
nx run ghost-monorepo:lint:boundaries |
✅ Succeeded | 20s | View ↗ |
nx run-many -t lint -p ghost,ghost-monorepo |
✅ Succeeded | 18s | View ↗ |
nx run @tryghost/admin:build |
✅ Succeeded | 7s | View ↗ |
nx run-many --target=build --projects=tag:publi... |
✅ Succeeded | <1s | View ↗ |
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗
☁️ Nx Cloud last updated this comment at 2026-08-11 20:54:38 UTC
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## data-db-types #29892 +/- ##
==============================================
Coverage 75.46% 75.47%
==============================================
Files 1607 1607
Lines 140789 140879 +90
Branches 17465 17468 +3
==============================================
+ Hits 106247 106327 +80
+ Misses 33494 33474 -20
- Partials 1048 1078 +30
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|

no ref
I recommend reviewing this one commit at a time.
This change should have no user impact.
This change:
In addition to automated testing, I also manually tested this. I sent a newsletter, opened it in Gmail, and verified that I saw the open appear in the newsletter's analytics.