nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
I ran into this problem and originally thought it was an issue with all the AI CKEditor options. I'm specifically using Summarize, Spell Check, Tone, and Generate with AI. As noted above I was trying to update another CKEditor setting and my edit screen wouldn't save (no error message), I looked at my AI CKEditor settings and noticed all my prompts had been removed.
When disabling and enabling the AI CKEditor Options the prompts didn't re-populate, I had to uninstall and re-install the AI CKEditor Integration module. Once I did that I had prompts entered for Summarize, Tone, and Spell Check. However, Generate with AI still didn't have a prompt and the field was required on checking enable. I applied a patch based on the MR above and I was able to enable the Generate with AI option and the prompt wasn't required.
I think this patch solves the immediate problem of the Generate with AI prompt field being required and would vote to move this to RTBC.
However, I think another issue should be created to provide the default prompt text for those other options if the option is enabled and the prompt field is empty. Would be interested to hear others thoughts on this, before the feature request is created.
seantwalsh → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
I would suggest someone here follow these steps → to take over this project. Hosting a Drupal 11 version of this on GitHub is not the best idea as it's outside of the Drupal Security and Regulatory ecosystem.
Added one comment, otherwise looks good.
Moving this to RTBC
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
Thank you!
Sorry my above comment had a typo and sounded really rude. I was supposed to say Gave not give. Turns out I can't give Credit. @jcandan can we give folks credit for working on this issue?
Give folks from above Credit for this issue. @jcandan any reason not to give folks credit?
After discussing this with some folks and security team members we are clear on why service accounts can't apply here. I am moving this to Close (works as designed) as we don't need to move forward with this at this time.
Created a MR based on the above patch. thought it might make getting this into a release a little easier.
johnpicozzi → made their first commit to this issue’s fork.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
jdleonard → credited johnpicozzi → .
I added Providence as a location and adding Group & Camp
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
I would agree this can be closed. I'm moving this to Fixed and giving credit to all involved.
Closing this issue as outdated as the 1.x version of this module only supports Drupal 8 & 9. We are in the works of releasing a 2.x version of the module that will support current versions of Drupal.
seantwalsh → credited johnpicozzi → .
Bumping this issue again. Looking to try to get the new version up before DrupalCon.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
We are moving forward posting our 2.0 version of this module and I need my co-worker added. Could you please also add Alexey Korepov (murz) as a co-maintainer on this project.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
I was here :-)
mandclu → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
froboy → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
Closing this and adding the speakers to the speaker issue
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
Ok, MR is now mergeable. Would be great to get this rolled in.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .