-
Notifications
You must be signed in to change notification settings - Fork 0
[$50] Prod - can't delete discussion #458
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
Comments
Challenge https://www.topcoder.com/challenges/5adcc53a-e371-4508-809c-335189a1584e has been created for this ticket. |
Challenge https://www.topcoder.com/challenges/5adcc53a-e371-4508-809c-335189a1584e has been assigned to obog. |
@jmgasper I've tested it in DEV and got the same error but I can delete discussions using REST API ('/discussions/{id}'): After deleting if you go to the discussion then the message should be displayed: I've looked at the code. The both controllers (DiscussionsController and DiscussionsAPIController) verify the current user's permission ('Vanilla.Discussions.Delete') in a category. They call the same method, but there are differences in the call parameters. I'll debug it today locally. |
@jmgasper DiscussionID is shown in URL ☝️ |
@atelomycterus - Done, deploying now, thanks. |
@jmgasper Verified in DEV. It works as expected. Normal user without 'Vanilla.Discussions.Delete' permission, copy&paste a discussion delete link in a browser address bar: |
Payment task has been updated: https://www.topcoder.com/challenges/5adcc53a-e371-4508-809c-335189a1584e |
Verified in Dev from Admin. |
@sdgun pls review my Prod screenshots and close ... |
Verified the screenshots. |
@atelomycterus - We're running into a problem on prod.
There was a spam post added that we want to delete, but no one seems to be able to. All permissions look fine for the admins, but it still gives us a 403, which is strange. We've tried clearing cache, logging out / in, and nothing seems to help.
The text was updated successfully, but these errors were encountered: