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.

Sign up for GitHub

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

Jump to bottom

✨ Add GDPR-Export for user #2444

Open
wants to merge 24 commits into
base: develop
Choose a base branch
from
Open

✨ Add GDPR-Export for user #2444

wants to merge 24 commits into from

Conversation

HerrLevin
Copy link
Member

@HerrLevin HerrLevin commented Mar 23, 2024 •
edited
Loading

  • Export all user data as zipped json-files
  • User-notification in notification-centre rather than email, showing both
  • UI for export
  • Configure Queue to not interfere with backup downtime Is this still needed?
  • Job to clean up profile archives
  • Adapt export to newly added data fields
  • Restrict GDPR-Export to once every 4 weeks
  • Restrict to limited users for beta-testing

jeyemwey
jeyemwey reviewed Mar 23, 2024
View reviewed changes
app/Models/User.php Outdated Show resolved Hide resolved
app/Http/Controllers/GdprExportController.php Outdated Show resolved Hide resolved
app/Enum/ExportableColumn.php Outdated Show resolved Hide resolved
MrKrisKrisu
MrKrisKrisu reviewed Mar 24, 2024
View reviewed changes
app/Models/OAuthClient.php Show resolved Hide resolved
app/Models/Status.php Show resolved Hide resolved
app/Models/User.php Show resolved Hide resolved
app/Models/User.php Outdated Show resolved Hide resolved
config/personal-data-export.php Show resolved Hide resolved
@HerrLevin HerrLevin marked this pull request as ready for review November 4, 2024 23:54
@HerrLevin HerrLevin requested a review from MrKrisKrisu November 4, 2024 23:54
@HerrLevin HerrLevin mentioned this pull request Nov 4, 2024
Open
MrKrisKrisu
MrKrisKrisu requested changes Nov 15, 2024
View reviewed changes
Copy link
Member

@MrKrisKrisu MrKrisKrisu left a comment

Choose a reason for hiding this comment

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

Wichtig wäre mir tatsächlich, dass wir die exportierten Daten fest definieren, der Rest ist non-blocking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MrKrisKrisu MrKrisKrisu MrKrisKrisu requested changes

@jeyemwey jeyemwey jeyemwey left review comments

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@HerrLevin @jeyemwey @MrKrisKrisu