"powered by klaro" info needed?

The text can be removed on the settings page technically: /admin/config/user-interface/klaro/texts

But I agree we should check if that's fine with the BSD license: https://github.com/klaro-org/klaro-js?tab=License-1-ov-file#readme

We might add a sentence in the description that it's friendly to keep it, but optional, otheriwse?

I created an issue over at the Klaro JS github issue queue: https://github.com/klaro-org/klaro-js/issues/526 will update here, if we get some more details.

See commented config.js in github:

    // You can also remove the 'Realized with Klaro!' text in the consent modal.
    // Please don't do this! We provide Klaro as a free open source tool.
    // Placing a link to our website helps us spread the word about it,
    // which ultimately enables us to make Klaro! better for everyone.
    // So please be fair and keep the link enabled. Thanks :)
    //disablePoweredBy: true,

@jan kellermann I'd vote to put that text with the link directly into the description of the field?

There is an answer: https://github.com/klaro-org/klaro-js/issues/526

• You can remove the "Powered by Klaro" message from the UI.
• You must keep the BSD-3 license text and attribution in the source code.
• Providing recognition elsewhere (e.g., documentation, config page) is a good practice but not legally required in the UI.

So we can add an option under styling.

yes, also spotted that and I read it the same!
Would be gr8 if we remove it.
thank you!

FYI: Initially it was build so that when under configuration->texts->consent dialog -> Powered By nothing is entered, it would not show the powered by, This seem to not work for a long time (since klaro_js upstream made a change that would display a "translation missing" if that was empty)

So i made a simple fix putting that back to work, the setting `disablePoweredBy` will be set to TRUE if there is nothing entered in the texts config, please test and review my change

I just the changed the form field description (you cannot "uncheck" a text input field).

But I would prefer an option to hide "Powered by" message because on multilangual sites you have to delete for every language.

Attaching a patch from the MR for use with composer.

FWIW this is working well for me in its current state, though I can see how @jan-kellerman's suggestion would make for better UX.

As suggested, i added a checkbox under "Settings->Styling" to set the library config "disable_powered_by" and reverted the changes to the text setting. Please review

Thanks @sascha_meissner - I added disable_powered_by: false default value. Guess attribution should be given by default?

Thank you @anybody - yes, disabling should by disabled by default :)

Did you test for RTBC?

Thx @anybody, correct, attribution should be given by default. AFAIR i was too lazy to add it to install as the result would be the same ;) thx for making this complete. I tested this throughly with mates so i am gonna set this to RTBC now to get this merged :)

The MR is accepted.

Thank you all for resolving this issue.

Automatically closed - issue fixed for 2 weeks with no activity.