Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: consent checkbox parameters #401

Merged
merged 2 commits into from
Jun 20, 2024
Merged

feat: consent checkbox parameters #401

merged 2 commits into from
Jun 20, 2024

Conversation

emmdurin
Copy link
Contributor

@emmdurin emmdurin commented May 7, 2024

Parameters for consent checkbox on account creation
See console PR#4239

Copy link
Contributor

@f-necas f-necas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it can be merged as main PR is merged :)

@fvanderbiest
Copy link
Member

fvanderbiest commented May 14, 2024

I'd rather rename these 2 options like this:

consent.agreement.activated => data.processing.agreement.activated
consent.agreement.url  => data.processing.agreement.url

because consent is too vague, what is really important here is you agree that your own data is processed according to the document for which a link is provided

@fvanderbiest fvanderbiest added this to the 24.0 milestone May 14, 2024
@f-necas
Copy link
Contributor

f-necas commented Jun 20, 2024

Changed in georchestra/georchestra:24.0.x branch

Copy link

💚 All backports created successfully

Status Branch Result
docker-master

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

github-actions bot added a commit that referenced this pull request Jun 20, 2024
[docker-master] Merge pull request #401 from georchestra/consent-checkbox
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants