- Issue created by @yautja_cetanu
- 🇮🇳India Vivek Panicker Kolkata
Asked the Deepchat maintainer about having consent modal here https://github.com/OvidijusParsiunas/deep-chat/issues/321.
- 🇩🇪Germany jurgenhaas Gottmadingen
As far as Drupal CMS goes, we don't need the DeepChat to implement anything regarding consent management. The Drupal Klaro module already implemented everything needed, we're just testing it.
What's important though, the DeepChat client loads Google fonts, which needs to be avoided. The Klaro maintainers have already opened an issue about that.
- 🇩🇪Germany jurgenhaas Gottmadingen
The Google font issue is at 📌 Remove Google Fonts from deepChat.bundle.js Active
- 🇩🇪Germany jurgenhaas Gottmadingen
This is being worked on at 📌 Add servives for AI features Active and I'll ping you here when it's ready for testing. Shouldn't be long, stay tuned.
- 🇩🇪Germany jurgenhaas Gottmadingen
This is ready for testing and review. Please note, the MR contains a couple of things that will be removed later on when everything is tagged in releases, especially the klaro config entities will be removed again later and the composer.json in the privacy recipe will be built back later as well.
But to give you an idea on the integration works and looks like, checkout the MR, run a
composer update
and then do a fresh site install. When you then apply the AI recipe and configure a provider, you will see how consent management for both the chatbot as well as alt text generation works. - 🇩🇪Germany jurgenhaas Gottmadingen
I've updated the MR and removed the temporary klaro config entities as they are now in Klaro's latest dev release.
- 🇺🇸United States phenaproxima Massachusetts
That MR looks straightforward and although the commit is blocked on getting a new tagged release of Klaro, I see nothing to complain about here.
- 🇩🇪Germany jurgenhaas Gottmadingen
@phenaproxima sure enough. There will not only be a new tagged release but maybe even a stable release of Klaro not soon from now. But let's keep this MR open anyways until the AI team had a chance to give us feedback. I will then add our typical tests there too.
- 🇩🇪Germany marcus_johansson
Thank you jurgenhaas and jan kellerman - after some discussions on Slack and in a thread here 🌱 Solution for checking localStorage Active , all is looking good for me. The descriptive texts are a little bit short, but I saw on Slack that this is not to think about yet.
We will release a new version of the AI module that will take care of the inherent issues outside of this issue soon.
- 🇩🇪Germany jurgenhaas Gottmadingen
This is great news, thanks @marcus_johansson
Next steps: I'll clean-up the MR when the next Klaro release tag will be available, hopefully later today, and then I write some tests. After that, it can be merged.
- 🇩🇪Germany jurgenhaas Gottmadingen
All tasks completed, tests added and all green.
-
phenaproxima →
committed 9a51704a on 1.x authored by
jurgenhaas →
Issue #3494518 by jurgenhaas, vivek panicker, marcus_johansson: Add...
-
phenaproxima →
committed 9a51704a on 1.x authored by
jurgenhaas →
-
phenaproxima →
committed c74b4869 on 1.0.x authored by
jurgenhaas →
Issue #3494518 by jurgenhaas, vivek panicker, marcus_johansson: Add...
-
phenaproxima →
committed c74b4869 on 1.0.x authored by
jurgenhaas →