-
Notifications
You must be signed in to change notification settings - Fork 33
Add the Content Summarization UI #147
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
base: develop
Are you sure you want to change the base?
Add the Content Summarization UI #147
Conversation
…not set' instead of an empty string if something isn't set, like the title or slug. Seems to give better results that way
…ty values in our helper function
…s that to our system instructions
…re in our prompt, making it clearer to the LLM what it should be summarizing
…ur request if it isn't empty. Change slightly how we separate content from context
… the description dynamic
…t from that to update our state. And when inserting the summary block, replace the existing one or insert at the top if there isn't an existing one
…here it's needed. Use this in the block controls regenerate button
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #147 +/- ##
=============================================
+ Coverage 46.89% 48.77% +1.87%
- Complexity 208 239 +31
=============================================
Files 19 22 +3
Lines 1271 1425 +154
=============================================
+ Hits 596 695 +99
- Misses 675 730 +55
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@dkotter perhaps a docs page for this experiment like done in https://github.com/WordPress/ai/pull/155/changes#diff-f70665388390657902e73741dea03efcdadc4bd0ceb4432959f8903ae015c042? |
|
Note we'll want to update the screenshot numbers when this gets merged in, I left it with a hash instead of a number depending on when this versus other experiments get merged in. |
Important
The code here is branched from #136.
Once that PR is merged, I will rebase this and then open this for review.
What?
Closes #12
The base Content Summarization Experiment and Ability were added in #136. This PR builds on top of that to add in the actual UI to trigger content summarization within the post editor.
Why?
With #136 merged in, you can manually trigger summarization by using the Ability but there isn't a way to trigger that from the UI. This PR adds in that UI, making that Experiment more useful.
How?
Testing Instructions
Settings > AI Credentialsand ensure you have a valid AI provider configuredSettings > AI Experimentsand ensure you have the Content Summarization Experiment enabledGenerate AI Summarybutton in the Post Status sidebarScreenshots or screencast
Test using WordPress Playground
The changes in this pull request can be previewed and tested using this WordPress Playground instance:
Click here to test this pull request.