Open rlmoser99 opened 1 month ago
Thanks for writing this up @rlmoser99. We're got two ban flows atm with both old and new admin operating concurrently. I'll put this on hold until we decommission old admin, that should happen within the next couple of weeks.
Theres two ways going about this...
We don't ban users that often, and theres a good chance they know why they've been banned. We could send a generic ban email with instructions to reach out to the contact email if they'd like find out more / appeal it.
This would be the quickest and easiest way to do it. We'd just need a mailer and hook that into the ban action.
A custom message is better, but also a lot more work that might not be worth it for the frequency of bans we have.
The UI will be the trickiest part. This is how we choose actions to perform against flags on admin v2:
We'd have to do something like Githubs review UI to include a message with a ban:
Thanks for taking the time to write up the two options! You are right, we do not ban very often, so the first route might be the best option. We could add a list of possible reasons for the ban in the email that we send.
It seems difficult to fit ban reasons into the existing UI, but I'm probably not being imaginative enough 😆
Where in the flag flow do you see ban reasons being slotted in?
Sorry for being unclear. I was thinking in the email body of the generic ban email.
We could send a generic ban email with instructions to reach out to the contact email if they'd like find out more / appeal it.
Sorry, thats my bad, you were clear, I just misread. I think that could work really well, it covers all the bases 💪
Hey @rlmoser99 this is unblocked and ready to work on. Would you like to update the acceptance criteria to match what we talked about with including reasons in the email?
For most of these stories, I've been writing acceptance criteria like this. The exact steps should better help demonstrate the flow we want and reduce any ambiguity.
@KevinMulhern When you get a chance can you check out the updated AC? Do you think it would be useful to list Technical considerations, like where the email template files should go or point to another example of the email?
AC looks great to me @rlmoser99 💪
Do you think it would be useful to list Technical considerations, like where the email template files should go or point to another example of the email?
Yeah it could be, if you think it will help I'd say go for it. Our user mailer is probably the best example, but the template might not be the best to follow haha.
Checks
Feature Request: brief description of feature request
format, e.g.Feature Request: Add a dark mode to the website
Description of the Feature Request
When an admin bans a user, the user should receive an email to let them know their account was banned, possible reasons, and an invitation to contact us.
Acceptance criteria
Email Body:
Additional Comments
No response