chore(release): v5.8.2 pins + changelog — week one on the wizard (gh-#745, gh-#722, gh-#724) - #769
Merged
Merged
Conversation
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Release chore for v5.8.2 · PLAN T467 · the #748 shape.
✅ What changed
compose.pinned.yaml: every imagehome-v5.8.1→home-v5.8.2.CHANGELOG.md: the v5.8.2 entry — gh-Ads: approved spots render one per 10-minute worker tick — approving several in the wizard looks like nothing happened #745 (approved spots drain every tick, Approve names the render window), gh-Ads: retiring a spot through the API leaves its media row eligible, so the retired spot keeps airing #722 (retire takes the spot off the air), gh-Ads API: a failed write/preview job loses its kind — the wizard cannot tell which step failed #724 (a failed job says which step failed, db/47), plus the merge-guard hook path (gh-toolkit: add a MERGE RULE PreToolUse guard (block gh pr merge, git tag, push to main) #751). Follow-ups noted: gh-admin-ui: toasts never mount on the production build (dev server is fine) #765, gh-ads: one shared retirer for the controller and the worker's stale sweep #767. Abstractions unchanged (package diff vs 5.8.1 empty).tools/check-doc-drift.shgreen: pins all on home-v5.8.2.🗄️ Upgrade note
db/47 is additive (
ADD COLUMN IF NOT EXISTS station.ad_spot.job_failed_kind), applied bymigrate.shon the way up like every other migration. No backfill: a job that failed before the upgrade keeps its error but shows on neither wizard step until the next job runs. setup.sh's marker check will say "db/47 adds no new table" — expected.🚦 Order
git tag v5.8.2 && git push origin v5.8.2(Dean's word, per the release rule).git pull && ./launch.sh --pinned.Do not merge without Dean.