Skip to content

fix: preserve dashboard migration sites on deploy - #337

Merged
kptdobe merged 1 commit into
mainfrom
fix/preserve-dashboard-migration-sites
Nov 5, 2026
Merged

kptdobe merged 1 commit into
mainfrom
fix/preserve-dashboard-migration-sites

Conversation

@kptdobe

@kptdobe kptdobe commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Fix

Production and stage HLX6_MIGRATION_SITES are dashboard-managed. The existing top-level keep_vars=true preserves dashboard variables only when deployment config does not explicitly override them.

Remove the empty declarations from production and stage; retain dev and test defaults. Document ownership and restoration after the previous reset.

Validation

Wrangler deployment dry runs passed for production and stage, with no HLX6_MIGRATION_SITES binding supplied by the deployment config. No live deployment or dashboard changes performed.

The previously cleared value still needs restoring in the dashboard.

Omit production and stage migration vars so keep_vars retains the dashboard-managed values instead of overwriting them with empty lists.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Nov 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@kptdobe
kptdobe merged commit 47c01ea into main Nov 5, 2026
6 checks passed
@kptdobe
kptdobe deleted the fix/preserve-dashboard-migration-sites branch November 5, 2026 13:42
adobe-bot pushed a commit that referenced this pull request Nov 5, 2026
## [1.16.2](v1.16.1...v1.16.2) (2026-11-05)

### Bug Fixes

* preserve dashboard migration sites on deploy ([#337](#337)) ([47c01ea](47c01ea))
@adobe-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 1.16.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants