Skip to content
This repository was archived by the owner on Sep 9, 2025. It is now read-only.

Merge devel into master #298

Merged
LorenzoJokhan merged 13 commits intomasterfrom
devel
Apr 20, 2023
Merged

Merge devel into master #298
LorenzoJokhan merged 13 commits intomasterfrom
devel

Conversation

@LorenzoJokhan
Copy link
Contributor

@LorenzoJokhan LorenzoJokhan commented Apr 20, 2023

Description

Tijdelijk op draft om te kijken wat er in devel zit wat een merge wellicht onveilig kan maken

Please include

  • Added support for concept idea created/published email
  • Safeguard against removing a site while there are still users active on it
  • Added tag type column

Issue reference

Fixes # (issue)

Type of change

Is it a new feature, bug fix, code improvement, etc.
If it is a breaking change what needs to be done to fix that

Documentation

Is the documentation updated, maybe a link

Tests

(How) has the change been tested

Branch

If the branch to merge to is not development

LorenzoJokhan and others added 13 commits January 16, 2023 11:30
…ted emailtemplates and added optional subject
…ion-to-deleting-sites

Add check on existing users bofore deleting a site
Added a migration to add type column and exposed it
* Rework classes to anonymize users in the background

* remove console.log statement

* Remove task code which does not really seem usefull in any way

* Remove the do check inside method doAnonymizeAllUsers, it will always be in 'do' replace req param with self

* Remove dangling console log statement
* Added conceptMail and conceptToPublishedEmail config

* Remove accidental .env
@LorenzoJokhan LorenzoJokhan marked this pull request as ready for review April 20, 2023 08:48
@LorenzoJokhan LorenzoJokhan merged commit a457dbf into master Apr 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants