"Contact Us" emails should only go to institution admins when sent by an institution member

Bug #800511 reported by Kristina Hoeppner
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Wishlist
Unassigned

Bug Description

Any admin on a site receives Contact Us emails. However, in order to have a better workflow and to know who is doing what, it would be better if institution admins received only Contact Us messages that were sent by members of their institution instead of any message. Many messages may pertain to things that they don't know about and thus should not receive messages as then nobody feels responsible for them.

Thus:
Contact Us message from logged out user -> Site Admin
Contact Us message from logged in user No institution -> Site Admin
Contact Us message from logged in user with institution -> Institution admin / admins if in multiple institutions

I would propose not to send the latter message to the site admin to avoid that admins don't know who also got this email / if somebody else already dealt with this.

I would also update the sentence at the bottom to if possible to make it clear why they get an email:

YYou receive this message because you are an administrator for the institution in which this person is a member. This is an auto generated notification from [site name]. To update your notification preferences, visit URL.

The same should go for Objectionable Content, and the virus messages. Though for the latter, the site admin should probably receive all emails.

Tags: newfeature
Changed in mahara:
importance: Undecided → Wishlist
status: New → Triaged
tags: added: contactus
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

There's also Bug 767569 that suggests some more granular improvements.

Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

"Contact us" messages are only sent to site admins and not institution admins. This must have been a bug at the time that got fixed.

Changed in mahara:
status: Triaged → Invalid
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

Re-opening this as sending 'Contact us' messages to institution admins from institution members that log in is important for multi-tennanted sites. We are going to ignore other messages and focus on the 'Contact us' messages in this wishlist item.

We already have a function that differentiates between institution admins and site admins when a person registers for an account and 'Confirm registration' is turned on. If there is an institution admin, they receive the notification, if there isn't, the site admin receives it. I would like to use the same logic here just for people who have an account as outlined above.

Changed in mahara:
status: Invalid → Confirmed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/12031

Changed in mahara:
milestone: none → 21.10.0
status: Confirmed → In Progress
tags: removed: contactus
description: updated
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/12031
Committed: https://git.mahara.org/mahara/mahara/commit/232ce9bbf5e3ab9e0944a732fb606013380ccced
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 232ce9bbf5e3ab9e0944a732fb606013380ccced
Author: Dianne Tennent <email address hidden>
Date: Thu Sep 23 17:16:33 2021 +1200

Bug 800511 directing 'Contact us' emails

If a person is logged in and is a member of
an institution, their 'Contact us' email is
sent to their institution admin.

If they do not have an institution admin,
it is sent to the site admin.

If they are not a member of an institution
or they are logged out, it is sent to
the site admin.

Email message footer has been updated, and
institution admins can adjust their
'Contact us' email settings.

Change-Id: Ia55feb53a7001d6bb3069464c082d29209874c53

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "21.10_DEV" branch: https://reviews.mahara.org/12054

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/12054
Committed: https://git.mahara.org/mahara/mahara/commit/6920141ed996b97925c0225aad710b71288f94b4
Submitter: Robert Lyon (<email address hidden>)
Branch: 21.10_DEV

commit 6920141ed996b97925c0225aad710b71288f94b4
Author: Dianne Tennent <email address hidden>
Date: Thu Sep 23 17:16:33 2021 +1200

Bug 800511 directing 'Contact us' emails

If a person is logged in and is a member of
an institution, their 'Contact us' email is
sent to their institution admin.

If they do not have an institution admin,
it is sent to the site admin.

If they are not a member of an institution
or they are logged out, it is sent to
the site admin.

Email message footer has been updated, and
institution admins can adjust their
'Contact us' email settings.

Change-Id: Ia55feb53a7001d6bb3069464c082d29209874c53
(cherry picked from commit 232ce9bbf5e3ab9e0944a732fb606013380ccced)

tags: added: newfeature
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.