- Issue created by @dstorozhuk
- π¨π¦Canada dstorozhuk Chicago πΊπΈ, Toronto π¨π¦, Kyiv πΊπ¦
- πΊπΈUnited States NicholasS
I too think this is how the module should have been setup, All the reports be editable and installed from config when first installing the module, and maybe a library of "Pre-made" prompts people can choose from so those default ones can be improved over time.
- π¨π¦Canada dstorozhuk Chicago πΊπΈ, Toronto π¨π¦, Kyiv πΊπ¦
I will be working on this feature possibly this week in scope of our project. So hopefully next week we have something.
- πΊπΈUnited States NicholasS
Great I can help if you get a branch going, I was going to maybe take a stab at it as well.
- π¨π¦Canada dstorozhuk Chicago πΊπΈ, Toronto π¨π¦, Kyiv πΊπ¦
I am not going to start at least today, so if you you like you can start.
The only think I want to push - I made some fixed in form it self, because it was bugy. Hovewer, it is not part of that issue, so I think we will just merge conflicts once we have one or another issue merged. - π¨π¦Canada dstorozhuk Chicago πΊπΈ, Toronto π¨π¦, Kyiv πΊπ¦
While the team and I and worked on the task https://www.drupal.org/project/ai_seo/issues/3536408 β¨ Make report type configurable Active , we decided to move it into an own module because the same code can cover many more use cases than just SEO.
I prompt everyone to switch to https://www.drupal.org/project/ai_content_advisor β since it is more flexible in creating new prompts types, and we have some ideas for future features that will be useful for others.
- π¨π¦Canada dstorozhuk Chicago πΊπΈ, Toronto π¨π¦, Kyiv πΊπ¦
Some fixes and new features in a new module:
- In a new module I also fixed the issue when for Anonymous users some HTML from Admin theme and logged-in users can show up.
- Also, fix a lot of small bugs in the code, specifically related to AJAX on the Analyze form.
- Added a block to show the Analyze form on the node page.
- Add a field to the report to show the used HTML, this can be useful for prompt results analysis.