Repository navigation
Turn off the Copilot chat feature flags - #6590
Draft
pawelangelow wants to merge 1 commit into
Draft
pawelangelow wants to merge 1 commit into
pawelangelow wants to merge 1 commit into
Conversation
Set documentationChat and databaseChat to false and bump the features config version to 13. The Copilot panel is hidden once neither chat is available, so the whole Copilot disappears from desktop builds. Installed apps pick the change up when they sync the config from main. References: #RI-8402
Contributor
Code Coverage - Backend unit tests
Test suite run success3924 tests passing in 331 suites. Report generated by 🧪jest coverage report action from 443d377 |
Contributor
Code Coverage - Integration Tests
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Sets
documentationChatanddatabaseChattofalseinredisinsight/api/config/features-config.jsonand bumps the configversionfrom 12 to 13. The Copilot panel is hidden once neither chat is available, so this removes the whole Copilot from desktop builds (both flags are limited toELECTRONbuilds, so Docker is unaffected).What will happen once this is merged
mainon GitHub about every 24 hours (RI_FEATURES_CONFIG_SYNC_INTERVAL) and only accept it when itsversionis higher than the one they already have, hence the bump.cloud-copilot-service, because apps will stop calling it.To revert: set both flags back to
trueand bumpversionto 14.Open question: I do not know whether the "Query Builder" feature behind the retired model is
databaseChatalone. If only that chat should go,documentationChatcan staytrue, but the Copilot panel will then still be shown.This is a draft because merging changes live behavior for every desktop user on every version. It should wait for a go from product.
Testing
features-config.jsonis valid JSON and the feature module unit tests pass (9 suites, 109 tests).SidePanels.tsxcloses the Copilot panel when neither chat flag is available. Please check the Copilot is gone on a desktop build before merging.Refs #RI-8402