-
Notifications
You must be signed in to change notification settings - Fork 47
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
Proposal: Flagging #61
Comments
If a flag was rejected it could affect the flaggers karma #9 |
@Wonderbread awesome, thanks! let's just do a single "Reason" text field, then have it posted as an AuditLog record, with we could even reuse the audit log form (AuditLogsController#edit) and redirect the user to it after flaggin |
basically the simple flow would be:
we can then update the audit log controller auth to allow the owner of that audit log to be able to edit it (instead of just checking for "moderator") |
Should all reports go in the audit logs? I feel like that could weigh them down a lot and important things could 'get lost' in the mess. They way I'm going about it right now is as follows:
It's less complicated than the long list makes it seem :) |
allow anyone logged in to flag a project (not just moderators)
moderators would be able to hide a flagged product (with a reason)
The text was updated successfully, but these errors were encountered: