GrafeasGroup / blossom

The website. The app. The everything.
6 stars 3 forks source link

Add button to app to report posts without claiming them #283

Closed TimJentzsch closed 2 years ago

TimJentzsch commented 2 years ago

Relevant issue: Close #259

Description:

This adds a second button to the transcription cards which allows the users to report a post without claiming it first.

Screenshots:

The new transcription card with an additional "Report" button.

The modal that opens when you click "Report". It allows you to select the report reason, namely that the post breaks the rules, that the post was removed or that OP already transcribed the post.

Testing Instructions:

Go to https://thetranscription.app/app/ and you should see a "Report" button for each transcription card. When you click the button, it should show a modal to choose the report reason. When you click the report reason it should send the report to Slack and then reroll the posts.

Future Work

Instead of rerolling all posts, it should only reroll the post you reported. That way, if you see multiple rule violations you could report all of them.

Checklist:

TimJentzsch commented 2 years ago

I tested this locally and it seems to work.

itsthejoker commented 2 years ago

I would like this to have a larger Transcribe! button than Report button on desktop and they should collapse down to full size buttons in mobile mode. Something like:

<div class="row">
  <div class="col-12 col-lg-8">
    <transcribe_button/>
  </div>
  <div class="col-12 col-lg-4">
    <report_button/>
  </div>
</div>
itsthejoker commented 2 years ago

Looks great :)