-
Notifications
You must be signed in to change notification settings - Fork 209
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
change: enable new chat and search UIs by default #2079
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline chat and inline touch was already disable before, right? Not that we're upsetting people by taking it away with no option to bring it back :) Rest LGTM
Yea they were turned off by default and confirmed with product here to remove inline chat here: https://sourcegraph.slack.com/archives/C066F85TE80/p1701709918296539?thread_ts=1701709448.915889&channel=C066F85TE80&message_ts=1701709918.296539 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
- new chat enabled and works ✅
- don't see old UI flags anymore ✅
- Search UI on by default and works ✅
- don't see fixup view anymore ✅
CleanShot.2023-12-05.at.09.53.20.mp4
Fixes #2292 Previously, detecting pastes into the document required the inline chat controller to be listening for changes. Since that was removed in #2079, we no longer detected pasts. This brings this behavior back via a dedicated event listener we add in `main`. ## Test plan - Ask cody to gen code - Copy it - Paste it into a doc <img width="1132" alt="Screenshot 2023-12-12 at 11 22 37" src="https://github.com/sourcegraph/cody/assets/458591/d3c14317-b7ad-4270-a41f-30448781d149"> <!-- Required. See https://docs.sourcegraph.com/dev/background-information/testing_principles. -->
CONTEXT: https://sourcegraph.slack.com/archives/C066F85TE80/p1701709448915889
change: enable new chat and search UIs by default
experimentalChatPanel
andexperimentalSearchPanel
configuration options.Follow up
Cean up chat manager and sidebar chat view.
Test plan
All the tests have been updated to run on the new chat UI instead.