Skip to content

Remove group from excluded_groups_list after delete#21017

Merged
MorrisJobke merged 1 commit intomasterfrom
enh/noid/clean-excluded-groups-list
May 18, 2020
Merged

Remove group from excluded_groups_list after delete#21017
MorrisJobke merged 1 commit intomasterfrom
enh/noid/clean-excluded-groups-list

Conversation

@kesselb
Copy link
Copy Markdown
Contributor

@kesselb kesselb commented May 17, 2020

Follow up to #19124. Remove group from shareapi_exclude_groups_list on delete. No backport required from my point.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels May 17, 2020
@kesselb kesselb added this to the Nextcloud 20 milestone May 17, 2020
@kesselb kesselb requested review from MorrisJobke and blizzz May 17, 2020 13:20
Copy link
Copy Markdown
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

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

Code looks good 👍

@MorrisJobke MorrisJobke merged commit 1bba7de into master May 18, 2020
@MorrisJobke MorrisJobke deleted the enh/noid/clean-excluded-groups-list branch May 18, 2020 12:24
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 technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants