- Issue created by @mostepaniukvm
- Merge request !2Issue #3465116: Add Chat Augmentor implemetnation. β (Merged) created by mostepaniukvm
- Status changed to Needs review
5 months ago 4:01am 1 August 2024 - πΊπ¦Ukraine mostepaniukvm
Moved a bit of code to base class and added Chat Augmentor implementation.
- πΊπΈUnited States j-barnes
@mostepaniukvm - Looks great, going to give this a try. Thanks for the MR.
- πΊπΈUnited States j-barnes
@mostepaniukvm - What are your thoughts on adding a role field and also incorporating the prompt replacement {input} similar to how the other augmentor behaves.
https://git.drupalcode.org/project/augmentor_chatgpt/-/blob/1.0.x/src/Pl...
if ($role == 'user') { $content = str_replace('{input}', '"' . addslashes($input) . '"', $this->configuration['prompt']); } }
- πΊπ¦Ukraine mostepaniukvm
Yes, it totally makes sense, pushed additional commit
- πΊπΈUnited States j-barnes
Added a couple minor tweaks, but tested and this is working great! Going to have some internal team members QA this and we should be good to go to merge. Thanks again for the contribution!
- Status changed to RTBC
5 months ago 7:06pm 2 August 2024 - πΊπΈUnited States Sunny-Lee
Working great! Short list of what was checked, have it enabled for pages:
- No issues changing the model through ECA
- Basic pages working no errors
- Pages with views no errors
- Pages with blocks and views no errors
- Generate button we have works, no errors
- Only able to test and see user role and it works well -
j-barnes β
committed 96fe7151 on 1.0.x authored by
mostepaniukvm β
Issue #3465116 by mostepaniukvm, j-barnes, Sunny-Lee: Implement Chat...
-
j-barnes β
committed 96fe7151 on 1.0.x authored by
mostepaniukvm β
- Status changed to Fixed
5 months ago 7:10pm 2 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.