Enable "Show example payload" buttons in GitOps mode - #46355
Conversation
We allow this already with "Preview payload" under Settings > Integrations > MDM > End user migration workflow.
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
There was a problem hiding this comment.
Pull request overview
Enables viewing “Show example payload” in policy automation modals even when GitOps mode is enabled, aligning behavior with other areas that allow payload preview without permitting configuration edits.
Changes:
- Removed the
gitOpsModeEnabledgating from “Show example payload” toggles in the “Other workflows” and “Calendar events” modals. - Added a changelog entry documenting the GitOps-mode UX fix.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| frontend/pages/policies/ManagePoliciesPage/components/OtherWorkflowsModal/OtherWorkflowsModal.tsx | Allows expanding the example payload in GitOps mode while keeping actual config inputs disabled. |
| frontend/pages/policies/ManagePoliciesPage/components/CalendarEventsModal/CalendarEventsModal.tsx | Allows expanding the example payload in GitOps mode while keeping configuration controls gated appropriately. |
| changes/fix-show-example-payload-gitops-mode | Adds user-facing changelog note for the GitOps-mode enablement. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Actionable comments posted: 0 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
WalkthroughThis PR fixes a bug where the "Show example payload" button was incorrectly disabled in GitOps mode for the "Calendar events" and "Other workflows" policy automation modals. The fix removes the Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #46355 +/- ##
=======================================
Coverage 66.89% 66.89%
=======================================
Files 2782 2783 +1
Lines 221734 221734
Branches 11221 11219 -2
=======================================
Hits 148334 148334
Misses 60000 60000
Partials 13400 13400
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@MagnusHJensen I merged this, then noticed #44719 has an "Awaiting QA" status. Should I not merge these until that's done, or is it safe to do so when GitHub allows me to? |
|
@spalmesano0 All good, I moved it to the board and put it to that status after approving with the assumption it would merge in reasonable time (which you did) so no worries. I could see it wasn't tagged with a group, so wanted to help you out. |



We allow this already with "Preview payload" under Settings > Integrations > MDM > End user migration workflow.
Related issue: Resolves #44719
Checklist for submitter
changes/,orbit/changes/oree/fleetd-chrome/changes.Testing
Summary by CodeRabbit
Bug Fixes