-
Notifications
You must be signed in to change notification settings - Fork 2k
[Odie] Bring back direct escalation when user downvotes or provided enough information #101949
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
[Odie] Bring back direct escalation when user downvotes or provided enough information #101949
Conversation
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
|
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: Async-loaded Components (~518 bytes removed 📉 [gzipped]) DetailsReact components that are loaded lazily, when a certain part of UI is displayed for the first time. Legend What is parsed and gzip size?Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. Generated by performance advisor bot at iscalypsofastyet.com. |
|
This PR modifies the release build for the following Calypso Apps: For info about this notification, see here: PCYsg-OT6-p2
To test WordPress.com changes, run |
sirino
left a comment
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.
I tested it, and it works as expected. I left a comment that I believe will be important to address. Please follow up on that.
| @@ -0,0 +1,22 @@ | |||
| import type { Message } from '../types'; | |||
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.
Using categories is a smart way to work around current tagging in the WP.com chat assistant. I think this is ok to merge so that we improve the user experience asap. But we should follow this up with further improvements to make sure we don't break the frontend by changing category names. Could you create a follow-up task for this in Linear and aim to address it next week? As far as I can see, it will require a change to the assistant yaml first.
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.
Yes, I agree, we should address this in ai-services
7bf0ade to
3ecfb83
Compare
Related to p1742588507341749-slack-C07D72S1135
Proposed Changes
We are bringing back direct escalation link and direct escalation when negatively rated, only in the case of user provided enough information. Remove the lingering code from the AB experiment.
Why are these changes being made?
There might be users that don't know they can ask to connect with human support
Testing Instructions
Pre-merge Checklist