[Highlighter] Highlighter settings update#6000
Conversation
jankalthoefer
changed the title
There was a problem hiding this comment.
Pull request overview
Updates the AI Highlighter installation experience in Settings, including a migration-gated install path (Streamlabs Replay) and a small UI/styling tweak for the migration carousel.
Changes:
- Route
installAiHighlighter()through a migration feature flag to either install Streamlabs Replay or use the legacy updater flow. - Update Settings’ Highlighter card to show a migration install carousel/progress flow when the migration is enabled and the app isn’t installed.
- Minor styling adjustment in the migration notice styles.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| app/services/highlighter/index.ts | Adds migration-gated install logic and adjusts install/download progress handling for Streamlabs Replay installs. |
| app/components-react/highlighter/SettingsView.tsx | Adds migration-aware installed-state checks and conditionally renders the migration installation flow vs. the legacy settings card controls. |
| app/components-react/highlighter/migration/MigrationNotice.m.less | Removes a background style from a migration notice element. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
BundleMonFiles updated (1)
Unchanged files (3)
Total files change +1.38KB +0.01% Final result: ✅ View report in BundleMon website ➡️ |
| style={{ width: 'fit-content' }} | ||
| type="primary" | ||
| onClick={() => { | ||
| HighlighterService.actions.installAiHighlighter(true, 'Highlighter-tab'); |
There was a problem hiding this comment.
Outside of scope nitpick: for analytics we're trying to move to typing the various strings in the usage statistics service to make it simpler to reference.
There was a problem hiding this comment.
But only for the Event names or also for every property sent to the analtics?
Uh oh!
There was an error while loading. Please reload this page.