Skip to content
This repository was archived by the owner on Oct 11, 2022. It is now read-only.

Conversation

@mxstbr
Copy link
Contributor

@mxstbr mxstbr commented Aug 16, 2018

Status

  • WIP
  • Ready for review
  • Needs testing

Deploy after merge (delete what needn't be deployed)

  • api
  • hyperion (frontend)
  • hermes

Todo

  • Add reportUser mutation to API
  • Add sendUserReportedEmail queue and add jobs to it from the mutation
  • Create user reported email template
  • Add button and form to frontend to report a user and add a message why you're reporting a user

@brianlovin can I get you to look at these last two tasks? Should be relatively quick to knock out since the template can just be plain text and the button and form on the frontend don't have to be anything fancy for this v1.

mxstbr added 2 commits August 16, 2018 14:34
Adds a job to a new `sendUserReportEmail` queue.
This isn't done yet, needs an email template.
@brianlovin
Copy link
Contributor

brianlovin commented Aug 17, 2018

@mxstbr wanna give my code a lookthrough as well as a local test? I sent you a test email through prod, so let me know if you're happy with that. Otherwise, the next iteration on this will be to start storing db records of reports and build a UI around it to take faster actions to ban bad actors as quickly as possible.

Other iterations:

  • make the report button accessible from a dropdown, so it's not so prominently exposed on user profiles
  • make the report button accessible from hover profile
  • add an option to send the report to community admins (if reported in-context of a community) or to the spectrum team
  • store reports in the db so we can easily:
    • look up all reports against a user
    • all users someone has reported
    • quickly take actions (ban) on a reported user

Copy link
Contributor Author

@mxstbr mxstbr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking great, let's ship this if tests pass! 🎉

@brianlovin
Copy link
Contributor

Roger that - admin merging :)

@brianlovin brianlovin merged commit 3d57bfd into alpha Aug 17, 2018
@brianlovin brianlovin deleted the report-users branch August 17, 2018 02:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants