-
Notifications
You must be signed in to change notification settings - Fork 40
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
… 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 47.93% 47.96% +0.03%
- Complexity 243 247 +4
=============================================
Files 22 22
Lines 1431 1453 +22
=============================================
+ Hits 686 697 +11
- Misses 745 756 +11
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.