Skip to content
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

Improve BlockedUserAlert emails #5776

Open
ragesoss opened this issue Apr 15, 2024 · 4 comments
Open

Improve BlockedUserAlert emails #5776

ragesoss opened this issue Apr 15, 2024 · 4 comments

Comments

@ragesoss
Copy link
Member

Current Behavior:

Wiki Education staff get alerts when a student's account gets blocked.

Desired Behavior:

If the block happens while the student is enrolled in a still-ongoing course, the student and instructor(s) should be emailed as well, with a full explanatory message in the email.

@cyrillefr
Copy link
Contributor

Hello @ragesoss ,

May I work on this issue ?

@ragesoss
Copy link
Member Author

ragesoss commented Apr 17, 2024 via email

@cyrillefr
Copy link
Contributor

Hello @ragesoss ,
I need clarification(s) :)

Currently, when creating a new Alert, there is only one course associated.
So, there is a mail sent to the content_expert of the last course of the student. (the one(s) who are greater).

But to be blocked is not associated with one particular course though I understand the mail sent to the last course content expert -> someone will get notice of the blocking.

So now, my guess is there should be the same number of Alerts that the number of ongoing courses the student is following. (3 ongoing courses -> 3 Alerts -> mail to 3 groups of instructors).

Also, should I then keep the mail to the content expert of the last course ? (def email_content_expert in app/modales/alert.rb. Are the 'greeter' the Wiki Education staff you are referring to ?

Do the mail sent to the student should be simple like 'You have been blocked in Wikipedia' or be implemented as an AlertMail or a regular mail ?

@ragesoss
Copy link
Member Author

I think keeping the current BlockedUserMonitor logic basically unchanged is fine. There might be a few rare cases where a student is in more than one current course, in which case the instructors for any courses except the latest one would not get an email, but I think that's okay. In practice, that will almost never happen.

Yes, the 'greeter' is the Wiki Education staff I was referring to.

If you implement an email with a simple message (but using the styling of the other emails that go to non-admins, like basically any email besides the AlertMailer one), I can expand it with additional text.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants