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

Delete calendar subscriptions as well when deleting user #28419

Conversation

tcitworld
Copy link
Member

Who forgot that... Oh wait, that's me. (#1545 was after #1197) 🙈

Closes #28418

Closes #28418

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@tcitworld tcitworld added bug 3. to review Waiting for reviews feature: dav php Pull requests that update Php code labels Aug 13, 2021
@tcitworld tcitworld added this to the Nextcloud 23 milestone Aug 13, 2021
@tcitworld
Copy link
Member Author

/backport to stable20

@tcitworld
Copy link
Member Author

/backport to stable21

@tcitworld
Copy link
Member Author

/backport to stable22

@tcitworld
Copy link
Member Author

Now that I think about it, should we add a repair job to delete subscriptions whose URIs don't match any existing users?

@szaimen
Copy link
Contributor

szaimen commented Aug 31, 2021

Merge?

@ChristophWurst ChristophWurst merged commit 613798a into master Aug 31, 2021
@ChristophWurst ChristophWurst deleted the bug/28419/delete-calendar-subscriptions-when-deleting-user branch August 31, 2021 06:59
tcitworld added a commit that referenced this pull request Sep 4, 2021
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
tcitworld added a commit that referenced this pull request Sep 4, 2021
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@tcitworld
Copy link
Member Author

Repair job in #28716

TSI-yogeshshejwadkar pushed a commit to nextmcloud/server that referenced this pull request Sep 14, 2021
deleteding an user

Follow-up to nextcloud#28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
backportbot-nextcloud bot pushed a commit that referenced this pull request Dec 1, 2021
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
backportbot-nextcloud bot pushed a commit that referenced this pull request Dec 1, 2021
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
MichaIng pushed a commit that referenced this pull request Dec 6, 2021
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
MichaIng pushed a commit that referenced this pull request Jan 12, 2022
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
MichaIng pushed a commit that referenced this pull request Jan 14, 2022
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
MichaIng pushed a commit that referenced this pull request Feb 14, 2022
deleteding an user

Follow-up to #28419

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug feature: dav php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Calendar subscriptions Survive User deletion
4 participants