WPB-22168 [fix-up] set the defaults of the cells feature correctly#4907
WPB-22168 [fix-up] set the defaults of the cells feature correctly#4907battermann merged 2 commits intodevelopfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR is a fix-up for WPB-22168 that corrects the default configuration values for the cells team feature flag. The title indicates this is a follow-up fix to ensure proper defaults are set.
Key Changes:
- Changes the
cellsfeature flag defaults from enabled/unlocked to disabled/locked - Updates the changelog entry to reference related PRs (#4889, #4907)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| charts/galley/values.yaml | Corrects cells feature defaults to disabled/locked, aligning with the pattern used by most other feature flags |
| changelog.d/2-features/WPB-22168 | Adds PR references to the changelog entry for traceability |
The changes look correct and align with the established convention in the codebase where most feature flags default to status: disabled with lockStatus: locked. The cells feature (the public-facing feature) is now properly configured to be disabled and locked by default, while cellsInternal (the internal feature) remains enabled and unlocked as intended. The changelog update appropriately links to the related PRs for traceability.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Checklist
changelog.d