-
-
Notifications
You must be signed in to change notification settings - Fork 17
Add tab for viewing and responding to reports in mbin. #195
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
Conversation
jwr1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I finally got around to reviewing this, so here we go :)
Hide accept/reject buttons on corresponding status. Add inkwell to navigate to reported item. Remove unnecessary ReportStatus values.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out Mbin does still use the closed report status, even though it should be approved. I'm going to create an issue on the Mbin repo for that, but I did create a temp fix here as well.
I also just fixed it so text wrapping would work (for small screens or long report reasons), and also made it so the flex direction of the report actions would change to horizontal if there's enough screen space.
Thanks!
|
I've just created an issue on Mbin about those extra report status types: MbinOrg/mbin#1639 |
Adds tab to mod panel for viewing and responding to reports.
Closes #36.