- Issue created by @kevinquillen
- πΊπΈUnited States kevinquillen
I committed a light POC here, but more work is required to complete this.
- Status changed to Needs review
8 months ago 10:47pm 29 July 2024 - Status changed to RTBC
8 months ago 7:34am 30 July 2024 - π¬π§United Kingdom scott_euser
Tried it out in a fresh install, works great - was able to carry out a number of tasks no issues. Nicely done!
- π§πͺBelgium wouters_f Leuven
One javascript widget to show them all!
That is a good improvement! Great work!
Thanks for this! - πΊπΈUnited States kevinquillen
Should the ckeditor plugin do a POST instead of a GET for security and content length (url) reasons? With a controller that returns an Ajax dialog form? Is that feasible? Would that break the ability to inject the response back to the editor?
- π³π±Netherlands jurriaanroelofs
Tested the latest commit and works great! Only thing that we can improve on in this mechanism is streaming the answer rather than waiting for completion of the messages before printing to the UI. It's what users are getting used to now, and it will also prevent time-out problems.
If I prompt the Completion plugin with "Write 2000 words about cats" I'll get "There was an error in the Summarize AI plugin for CKEditor." probably because of the time-out.
That error message is wrong but I addressed that in my MR here: https://www.drupal.org/project/ai/issues/3464266 π UI text inconsistencies and user experience issues of main assistant plugin Active
- πΊπΈUnited States kevinquillen
I don't know how to support streaming while also utilizing Drupal Form API due to how it handles AJAX command events.
- π¬π§United Kingdom scott_euser
Streaming definitely separate feature request like you raised here π [META] Implement Streamed responses in CKEditor plugins Active
So this one is good to merge I think! Thanks! -
kevinquillen β
committed c3c97168 on 1.0.x
Issue #3464475 by kevinquillen, scott_euser, wouters_f: Add new CKEditor...
-
kevinquillen β
committed c3c97168 on 1.0.x
- Status changed to Fixed
8 months ago 2:42pm 31 July 2024 - πΊπΈUnited States kevinquillen
Opened new issues for further development.
-
kevinquillen β
committed c3c97168 on automators-ux
Issue #3464475 by kevinquillen, scott_euser, wouters_f: Add new CKEditor...
-
kevinquillen β
committed c3c97168 on automators-ux
Automatically closed - issue fixed for 2 weeks with no activity.