humanswitch / consentcookie

Open source AVG solution for websites
Apache License 2.0
27 stars 10 forks source link

Descriptions based on settings #6

Open tubbynl opened 6 years ago

tubbynl commented 6 years ago

Description

Some details in the description need to be based on tracker settings on the website

Expected Behavior

The description "Google Analytics verzameld geanonimiseerd gegevens over het surfgedrag op deze website. Door het anonimiseren is dit niet naar jou te herleiden." should partially be based on the initstate setting.

"Door het anonimiseren is dit niet naar jou te herleiden." depends on how GA is configured (anonymise thingie).

Possible Fix

This could reflect as a option for 'initstate'

StevenChoo commented 6 years ago

Need to discuss it.

Several questions:

tubbynl commented 6 years ago

no, i think it should be a setting like "you can't disable it and i(webmaster) have set the settings of this tracker in such a way it's workings are anonimised"

being that the webmaster "says" he's done actions in the trackers config to ensure anonimity

being an explicit setting it's an webmaster's responsibility to do so

StevenChoo commented 6 years ago

I think one of the problems is that the state currently is called: disabled.

Something like "alwayson" would be more clear.

With that, adding a flag next to the initstate property, e.g. "anomymous":"true" would give the option to let ConsentCookie know the configured tracker/app is collecting data anonymously.

connection config example:

{
    "initstate": 1,
    "anonymous": true
}

With this you can also provide an option for optin / optout and set the flag anomymous = true In the "connections" if the flag is set, ConsentCookie will show a generic message e.g "This website uses this tracker to collect data anonymously"

(the used naming are a proposal and for debate)

tubbynl commented 6 years ago

yes, that is more explicit and transparent, i also agree on the naming, i assum the enum-values of initState remains are as they are used now (optin-optout-disabled)

StevenChoo commented 6 years ago

i assum the enum-values of initState remains are as they are used now (optin-optout-disabled)

Yes

optin = 1 optout = 0 alwayson = -1

StevenChoo commented 6 years ago

partially fixed by #15