Repository navigation
chore(release): approve core migration 0034 (ad_daily_stats) for production - #646
Closed
hikariming wants to merge 1 commit into
Closed
hikariming wants to merge 1 commit into
hikariming wants to merge 1 commit into
Conversation
…uction Adds 0034_ad_daily_stats.sql (from #644) to the production schema approval manifest so the production deploy applies it. Without it the deploy skipped the migration and every /api/sponsors/pulse write 500s. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
hikariming
requested review from
AsperforMias and
KurosawaGeeker
as code owners
November 5, 2026 18:58
Collaborator
|
Since 2026-11-07 ghfind is split into three repositories, and Verified in production (read-only, 2026-11-08):
Thank you for the precise diagnosis! / 已移植到私有后端仓库并随 v1.0.4 上线，生产库已执行 0034，赞助位曝光数据恢复写入。感谢！ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #644. The production deploy only applies migrations listed in
ops/feed-production-schema-release.json. 0034 wasn't listed, so step 12 skipped it: production D1 is still at0033and has noad_daily_statstable. The new code is live, so everyPOST /api/sponsors/pulsewrite currently returns 500.This adds the
0034_ad_daily_stats.sqlname + sha256, following the 0029 approval in beb6f23, and bumps the expected core count to 24.node --test scripts/feed-production-release.test.mjs: 20 pass.The migration is additive only: one new table, one index, one nullable column on
sponsorships, and 7UPDATEs settingadvertiser_id.🤖 Generated with Claude Code