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

Change: add settings for an explicit data privacy statement #375

Merged
merged 18 commits into from Jun 7, 2018

Conversation

auge8472
Copy link
Collaborator

@auge8472 auge8472 commented May 2, 2018

Tasks:

  • add the settings to the install script
  • add the settings row to the settings form
  • add the necessary strings for the settings form to the language files
  • add the code for procession of settings in the admin.inc.php
  • add the necessary fields in the user data table
  • add the code for accepting the data privacy statement in the registration form
  • add the code for accepting the data privacy statement in the posting form
  • add the code for accepting the data privacy statement (again) after a later introduction or a change
  • redirect the user to the agreement form when one has not accepted the agreement and tries to post an entry

@auge8472 auge8472 added this to the 2.4.10 milestone May 2, 2018
@auge8472 auge8472 self-assigned this May 2, 2018
auge8472 added 16 commits May 2, 2018 21:18
…epted the agreement and tries to send a posting
@auge8472 auge8472 requested a review from loesler June 5, 2018 21:12
@auge8472
Copy link
Collaborator Author

auge8472 commented Jun 5, 2018

All tasks done. I've tested the function of the parts in a more or less clean install. A review by @loesler is requested. There is no update function so far because we need two different functions for the branches legacy_2.4. and master anyway. I want to solve the mentioned conflict directly before merging as the last step.

@auge8472 auge8472 merged commit f79d752 into My-Little-Forum:master Jun 7, 2018
@auge8472 auge8472 deleted the gdpr branch June 7, 2018 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant