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

Remove deleted registration forms from ACLs #5131

Merged
merged 1 commit into from
Oct 26, 2021

Conversation

jbtwist
Copy link
Contributor

@jbtwist jbtwist commented Oct 25, 2021

I was not sure about how to name this commit and PR, since it seems verbose to me, but I tend to talk verbosely, so feel free to change the name of the commit and PR however you feel it is cleanlier.

@ThiefMaster ThiefMaster changed the title Fix Issue#5130 Remove deleted registration forms from ACLs Oct 25, 2021
@jbtwist jbtwist changed the base branch from master to 3.0.x October 25, 2021 16:38
@ThiefMaster ThiefMaster added this to the v3 milestone Oct 26, 2021
@ThiefMaster ThiefMaster enabled auto-merge (rebase) October 26, 2021 09:32
@ThiefMaster ThiefMaster added this to In progress in Release 3.0 via automation Oct 26, 2021
@ThiefMaster ThiefMaster merged commit df064f0 into indico:3.0.x Oct 26, 2021
Release 3.0 automation moved this from In progress to Done Oct 26, 2021
@jbtwist jbtwist deleted the Issue#5130 branch October 26, 2021 11:05
ThiefMaster added a commit that referenced this pull request Jun 22, 2023
Before #5131 got merged, registration forms were not removed from ACLs
when someone deleted the form. Cloning such an event would fail because
the registration form is not in the mapping (as deleted forms are not
cloned).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Release 3.0
  
Done
Development

Successfully merging this pull request may close these issues.

Registration Forms ACL entries are not deleted when deleting the registration form
3 participants