fix: update Content Health Monitor setup instructions for new Connect UI#345
fix: update Content Health Monitor setup instructions for new Connect UI#345
Conversation
The Connect settings drawer replaced the standalone "Vars" tab with an "Advanced" tab containing an Environment Variables section. Update the setup instructions and screenshots to match the current UI. Closes #344 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Extension release summary🚀 Will release on merge
See the contributing guide for details. |
jonkeane
left a comment
There was a problem hiding this comment.
We could just change all of these to new settings (might be another nudge to get folks to upgrade "look at all this new stuff"), but if this is rendering on the server, we should be able to detect which version of connect someone is on and show the old settings if they are on an older version and the new ones if they are on a newer one, yeah?
Good idea! Version detection is possible via the /v1/server_settings API (it returns a version field). The extension already has a Connect client, so we could query it during setup. The tricky part is that even within the same Connect version, individual users can toggle between old and new settings via the preview feature (userPreviewNewSettings), so server version alone doesn't tell us which UI they're seeing. Since minimumConnectVersion is already 2025.04.0 and the new drawer defaults to on, I think targeting the new UI is the right call here. The old settings pane will eventually be removed, and this gives users on the old UI a nudge to switch over. |
There was a problem hiding this comment.
this is the place to update in order to make sure your changes get released
Use client.version to show new UI instructions (Settings button, Advanced tab) on Connect >= 2026.03.0 and old UI instructions (gear icon, Vars tab) on older versions. Bumps version to 1.0.1. Closes #344 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Thanks @karawoo for the review and the version bump catch! I know you and @jonkeane were both leaning towards version detection, so I tried to implement it before settling on just targeting the new UI. Here's a summary of the options I explored — I've pushed Option 2 as a prototype, but my lean is Option 1. Happy to keep Option 2 if you all think the version detection is worth it. Option 1: New UI only (current/initial PR approach) ✅ Simple, no additional logic Option 2: Server version detection (prototype implementation for evaluation) ✅ No extra API calls (SDK already fetches /server_settings) Option 3: User preference detection (GET /me) ✅ 100% accurate |
Summary
The Connect UI was redesigned — the standalone "Vars" tab was replaced with an "Advanced" tab containing an Environment Variables section, and toolbar buttons are now labeled. The Content Health Monitor's setup instructions and screenshots were outdated.
Changes
vars.png,settings-gear-icon.png) with new ones (env-vars.png,settings-button.png, updatedrefresh-report.png)manifest.jsonwith new file entries and checksumsCloses #344
Note
There are similar (minor) stale UI references in other extensions — will file a separate cleanup issue for those.