-
Notifications
You must be signed in to change notification settings - Fork 0
[$50] Re-enable "Move" #384
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/1643d90a-ef6c-4d98-9aeb-6adee4194049 has been created for this ticket. |
Challenge https://www.topcoder.com/challenges/1643d90a-ef6c-4d98-9aeb-6adee4194049 has been assigned to obog. |
@jmgasper Wrote notes about Moving Discussions and re-enabled 'Move'. Please apply PRs:
UIAdded 'Cancel' button for consistency with other popups (Add Category/Invite user and so on) Permissions in Vanilla'Garden.Moderation.Manage' is a global Vanilla permission. Use it for moderator/admin roles. if User has 'Garden.Moderation.Manage'/'Groups.Moderation.Manage' and can edit a discussion then Moving DiscussionsThe next permissions will be checked after moving confirmation:
'Leave a redirect link'if 'Leave a redirect link' is checked then a discussion record (type= redirect) is added in Discussion table. It may be required for tracking some discussions. I moved the discussion (id=164) + checked 'redirect link': Email for a discussion (type=redirect): Moving content from a category |
Payment task has been updated: https://www.topcoder.com/challenges/1643d90a-ef6c-4d98-9aeb-6adee4194049 |
Verified in Dev unable to verify in prod since only admin users can perform this action. |
@atelomycterus - I ran into a situation where we may need to move some discussion around. We had originally just disabled it here: #117
Can we re-enable and just fix up that popup so it looks a little better please? My bad for not thinking further ahead, thanks.
The text was updated successfully, but these errors were encountered: