The attached patch converts user access rule deletions to the deletion API.

NB: There are other minor problems with the access rule forms, but they are unrelated and out-of-scope for this patch.

CommentFileSizeAuthor
#1 dapi_user_rules_0.patch2.76 KBhunmonk
dapi_user_rules.patch2.36 KBasimmonds

Comments

hunmonk’s picture

hunmonk commented
Priority: Normal » Critical
StatusFileSize
new dapi_user_rules_0.patch2.76 KB

code style looks good.

attached patch also fixes the bad query that prevented access rules from being added at all. the code in that section probably needs more work, but at least with this minimal fix you can actually test the patch... :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community

Works for me!

asimmonds’s picture

asimmonds commented
Status: Reviewed & tested by the community » Closed (won't fix)

The deletion API was rolled back.