-
Notifications
You must be signed in to change notification settings - Fork 8
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
Feature/dsgvo deletion #509
Open
saschasommer
wants to merge
394
commits into
redesign-dev
Choose a base branch
from
feature/dsgvo-deletion
base: redesign-dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
...in /account/deactivated/, and instead just show their own groups. The django admin interface should be used for viewing others' groups as admin
* redesign #318: remove "Edit my Profile" personal menu entry https://git.wechange.de/gl/code/redesign/-/issues/318 * redesign #321: move "set up my profile" button from menu to leftnav https://git.wechange.de/gl/code/redesign/-/issues/321 * Adjust test_navigation_api tests for changed menu structure * Also removed v2 profile menu buttons from #321 and #318 on v3 portals * Redesign #322: Move "Change language" from user menu to profile leftnav It appears only in v3 contexts and has the hardcoded id "ChangeLanguage" https://git.wechange.de/gl/code/redesign/-/issues/322 * redesign #311: v3 main content: re-order group-area leftnav buttons https://git.wechange.de/gl/code/redesign/-/issues/311 * v3 main navigation: add "Events" forum alongside forum For portals that have a split Forum/Events group * redesign #315: v3 navigation: Move "Discover" link - no longer shown in main navbar services for logged in users - still shown in main navbar services for anonymous in users - now shown with correct icon in spaces community section * v3 main menu: show Managed Tags Paired Groups by default (was not showing Portalpartner Forums else)
Brings the layout them in line with the other tiles.
Parsing the `target="_blank"` attribute of links.
No /account/ URLs should show UI pages, but should only be service-endpoints
Issue: https://git.wechange.de/gl/code/cooperation/-/issues/430 * Convert RocketChat group hooks to Celery tasks * Convert RocketChat relay message hooks to Celery tasks * Convert RocketChat user hooks to Celery tasks * Fix profile update hook Profile update is called without force_update and update_password. * Move group room id computation to a model helper * Remove unrequired group pre_save hook * Remove TODO No need to blocking creation of relay messages * Fix user activated hook * Add test for renaming a user with a colliding name * Fix rocket chat test object deletion Move all deletion code to tearDown to avoid leaving stale objects in RC after an assertion error or exception. * Fix relay message hook * Disable further celery email integration Temporary disabled deliver_mail_task. * Move conference room membership RocketChat hook to a Celery task
Issue https://git.wechange.de/gl/code/dina-customer/-/issues/831 * Add node version restriction Conference frontend does not compile with node >16 * Add button to open result project instead of embedding it The result embed iframe does not work with the redesign.
Issue: https://git.wechange.de/gl/code/cooperation/-/issues/428 * Add superuser promote / demote signals * Add hooks to make superuser Nextcloud admins fixup * Add admin action to make user Nextcloud admins * Add RocketChat hooks to admin handling
Allow access to authenticated users to BBB rooms in public events or groups
- refactor group deletion functions - directly send mail instead of using queued mail - email text improvements - send notifications on deactivation instead of on deletion - respect non-admin user notification settings for deactivation notification - send inactivity notifications only to admins - move user manual deletion email notification from the delete view into the deactivate function to be consistent with other notifications - run activity update at night
Update dateutil dependency
The ES threads load to data inconsistency during tests
Just to make sure not to break inactive portals
Add tests for inactive group deletion
Returns forum, events and default user groups. Considering that forum or events might not be included in the default user group in some setup.
Don't suggest contacting support for an earlier deletion
By using asser_called_once_with we make sure that no other notification were send
9df7fd4
to
b895ea4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Created a second pull request because #474 cannot be re-opened with a deleted base branch.