Removed the legacy Explore integration - #29880
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
It looks like this PR contains a migration 👀 General requirements
Schema changes
Data changes
|
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run ghost:test:ci:integration |
✅ Succeeded | 3m 3s | View ↗ |
nx run @tryghost/admin:test:acceptance |
✅ Succeeded | 7m 20s | View ↗ |
nx run ghost:test:integration |
✅ Succeeded | 3m 10s | View ↗ |
nx run-many -t test:unit -p @tryghost/admin,ghost |
✅ Succeeded | 4m 42s | View ↗ |
nx run ghost:test:legacy |
✅ Succeeded | 2m 53s | View ↗ |
nx run ghost:test:e2e |
✅ Succeeded | 2m 45s | View ↗ |
nx run ghost-admin:test |
✅ Succeeded | 2m 47s | View ↗ |
nx run ghost-monorepo:lint:boundaries |
✅ Succeeded | 22s | View ↗ |
Additional runs (3) |
✅ Succeeded | ... | View ↗ |
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗
☁️ Nx Cloud last updated this comment at 2026-08-11 15:19:28 UTC
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #29880 +/- ##
==========================================
+ Coverage 75.42% 75.46% +0.03%
==========================================
Files 1606 1594 -12
Lines 140657 140382 -275
Branches 17431 17377 -54
==========================================
- Hits 106095 105934 -161
+ Misses 33514 33379 -135
- Partials 1048 1069 +21
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 Explore Ping replaced the iframe-based pull integration, so Ghost no longer needs to expose a dedicated Admin API endpoint or retain its long-lived built-in credential.
no ref Removing the built-in Explore integration changed fixture counts and removed the endpoint used to verify core integration keys.
fc41cd2 to
f28c732
Compare

What changed
GET /ghost/api/admin/explore/endpoint and backend serviceghost-explorereferral attributionTesting
pnpm nx run ghost:build:tscpnpm nx run ghost-admin:lintpnpm nx run ghost-admin:buildpnpm nx run @tryghost/admin:buildUPDATE_SNAPSHOT=1 pnpm test:single test/e2e-api/admin/roles.test.jspnpm test:single test/unit/server/data/schema/integrity.test.jspnpm exec vitest -c vitest.config.db.ts test/integration/migrations/migration.test.js --runuptwice anddownonce