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

Can not stop teachers from receiving submission notifications #148

Closed
jtutchings opened this issue Mar 16, 2016 · 34 comments
Closed

Can not stop teachers from receiving submission notifications #148

jtutchings opened this issue Mar 16, 2016 · 34 comments

Comments

@jtutchings
Copy link

Turnitin Assignment 2 version 2016011104

Teachers are receiving submission notifications whenever a student submits to an assignment and there does not appear to be an option to turn this off. Also users who have a manager type role at a category level are being sent the emails.

Example email
Subject: Submission made to assignment

A submission entitled 2 has been made to assignment REN318LON CW1 Submission: Part 1 in the class REN318LON - International Hospitality Operations Management - 1516AAA.

Submission ID: 12345678
Submission Date: 14-Mar-2016 06:24PM

@aspark21
Copy link
Contributor

You can disable this by changing the default messaging outputs for "Turnitin Assignment Instructor Digital Receipt notifications" and deselecting it. Instructors can then enable it if they want to.

It probably should not be enabled by default.

@jtutchings
Copy link
Author

We have this setting disabled but our staff have still been getting the emails

@jmcgettrick
Copy link
Contributor

It shouldn't happen if - as @aspark21 says - you disable the instructor digital receipt notifications. If you disable emails but still keep popups enabled do the popups still send?

For reference, the plugin defines instructors as users who are able to grade so that will in reality encompass a number of roles.

@jtutchings
Copy link
Author

Thanks for your replies.
We have actually ended up setting it as "Disallowed" as all other configurations didn't stop the emails unfortunately I had a new report this morning that the emails are being sent out.

When I check the users message settings that are reporting the emails they have all been set not to receive the emails. My only option for the ones that have been receiving lots of the emails is to temporary disable notifications, which is not perfect as they will no longer get any messages out of Moodle.

@aspark21
Copy link
Contributor

aspark21 commented Apr 1, 2016

@jmcgettrick Same as @jtutchings for us too actually, we went from setting the default to not send, now to disallowed and we are still getting notifications being sent to teachers. Could it be the turnitin website sending them instead?

@aleksmacheta
Copy link

We just updated Turnitin Assignment 2 version 2016011104 in our Production environment this morning and are noticing the same issue.
We have had to 'Disallow' Turnitin Assignment Instructor Digital Receipt notifications. However our Instructors are still receiving notifications each time a student submits!

Our workaround has been to advise all Instructors to edit their messaging settings manually and untick pop-up and email notifications for:

Turnitin Assignment Digital Receipt notifications
Turnitin Plagiarism Plugin Digital Receipt notifications
Turnitin Assignment Non Submitter notifications

We don't want to untick these at a global level since students rely on getting email submission notifications.

@aleksmacheta
Copy link

In addition, the documentation (https://guides.turnitin.com/03_Integrations/Turnitin_Partner_Integrations/Moodle/Moodle_Direct_V2/Direct_V2_Integration_Administrator_User_Manual/Enabling_Student_Submission_Notifications) states that
"As an administrator, you can choose whether a notification is automatically sent to instructors when a student has made a submission to an assignment. This allows instructors to keep track of submissions and act on late submissions to an assignment. This setting is set to No by default. By enabling this feature, instructors will receive a Moodle notification, which can be accessed from their Moodle inbox.".
It actually seems to be set to YES by default.

@Syxton
Copy link

Syxton commented Apr 21, 2016

Another issue is the line in turnitintooltwo_submission.class.php ~line 630:
$this->submission_instructors = get_users_by_capability($context,'mod/turnitintooltwo:grade', 'u.id');

This will retrieve all users that have this ability at this context...however that includes site admins and managers. The better code would be:

$this->submission_instructors = get_enrolled_users($context,'mod/turnitintooltwo:grade', 0, 'u.id');

This then limits the returned users as having to have an active enrollment in the course and cuts off users that have system permissions that would satisfy the original code.

@rlorenzo
Copy link
Contributor

rlorenzo commented Apr 22, 2016

@Syxton A better solution might be to just email those listed as Tutors and not just get anyone with the grade capability. One of the developers at our University is working on such a patch.

But the list of tutors should only be those users with the capability to grade at the course level, right?

rlorenzo referenced this issue in ccle/moodle-mod_turnitintooltwo Apr 22, 2016
- modify submission notification list to only include turnitin tutors who
  is currently active moodle user
@ghost
Copy link

ghost commented Apr 26, 2016

We just experienced the same issue. Changing notification settings made no difference.

The list of message outputs defined for the plugin are (from https://github.com/turnitin/moodle-mod_turnitintooltwo/blob/v2016011105/db/messages.php):

  • submission
  • nonsubmitters
  • notify_instructor_of_submission

instructor_message.php gives the message name as submission, surely this should be notify_instructor_of_submission? See https://github.com/turnitin/moodle-mod_turnitintooltwo/blob/v2016011105/classes/digitalreceipt/instructor_message.php#L42.

@cwestgarth
Copy link

Hi @jbriggscmet, @rlorenzo, @aleksmacheta, @Syxton, @jtutchings, @aspark21

Thank you all for your input and raising this with us. I will pass this onto the Engineering Team to investigate further.

Regards
Caroline Westgarth
(Product Manager, Turnitin)

@VladimirSavitchi
Copy link

Hi, we've just updated to v2016011105 and found that notifications are being sent to all the tutors.
Regardless that the group of the student with the tutor is different.

@davidmhatton
Copy link

Hi @VladimirSavitchi, we are aware of the plugin behaviour whereby every tutor is emailed and are currently working on addressing it. thanks for reporting.

@haietza
Copy link

haietza commented Jun 15, 2016

A workaround for this is to leave the default message outputs for email configured to Permitted, then to disable the individual user message preferences for Turnitin Email (user profile -> administration -> preferences -> messaging).

@cb526
Copy link

cb526 commented Jun 21, 2016

Is there any progress on this? We would not have updated had we known of this issue.

@cwestgarth
Copy link

Hi @cb526,

We have fixed the issue but we have yet to release the version of the plugin containing the fix. We will look to release within the next month.

Thanks

@whymiss
Copy link

whymiss commented Jul 5, 2016

Please hurry. Many non-teachers listed under Other Users (site wide) are getting angry at hundreds of emails being received one by one from every course.

@aspark21
Copy link
Contributor

aspark21 commented Jul 5, 2016

@cwestgarth What's up with the release schedule? Usually there's a monthly release like clockwork but there hasn't been one in 3 months. Is it because of the work on the migration tool from V1?

@cwestgarth
Copy link

Hi @aspark21 the team here in the UK have been working on a broader project over the last couple of months. We are looking to release in the next couple of weeks which will include support for Moodle 3.1.

Thanks

@aspark21
Copy link
Contributor

aspark21 commented Jul 5, 2016

👍

sharpchi referenced this issue in sharpchi/moodle-mod_turnitintooltwo Jul 8, 2016
@sharpchi
Copy link

sharpchi commented Jul 8, 2016

Just want to add another scenario: for our department administrators we give them the grading capability (as this is the only way they can view TII submissions) at the category level (i.e. we assign "dept admin" role at the category).

manage category

They are not explicitly enrolled on the Moodle courses (i.e. not listed in the user list), but they still get an email. If a user is not enrolled on a Moodle course they shouldn't really be getting these emails.

@cwestgarth
Copy link

Hi @sharpchi,

Thank you for raising this with us. As we only have three roles in Turnitin we acknowledge anyone who has the permission to grade an assignment as an instructor. We therefore enrol them in Turnitin as an instructor. This is why they are receiving the email.

@sharpchi
Copy link

sharpchi commented Jul 11, 2016

Thanks for replying @cwestgarth, I understand what you're saying, the distinction we're making is that these people are not enrolled on the course, so perhaps they shouldn't be enrolled in Turnitin.

It's a mute point anyway, since updating TII in Moodle, we've had so many complaints about too many unwanted emails that we've nobbled the messages for instructors entirely.

@cwestgarth
Copy link

Thank you @sharpchi and apologies for the inconvenience.
I will create a ticket for our engineering team to look into further.
Many thanks.

@aspark21
Copy link
Contributor

@cwestgarth is the pull request for the fix public? I don't think I saw it but if not any chance it could be?
Or is there any chance the new version will be released by Tuesday?

We are scheduled to be upgrading our main site to 3.0 Wednesday, if this isn't ready we are going to have to cancel our upgrade...with little hope of having a new slot anytime soon.

I also had not realised that this would be triggering the processing of the email notifications backlog to teachers as other institutions have found out during their upgrades. We can't be going through that again.

@hitteshahuja
Copy link

@cwestgarth does this affect the plagiarism plugin too for moodle 3?

@cwestgarth
Copy link

Hi @hitteshahuja,
This affects the Plagiarism Plugin too.

Thanks

@hitteshahuja
Copy link

@cwestgarth Can you please confirm when this is due to be fixed ( even if on a "develop" branch which can be ported through) . We upgrade to 3.0.4 in a week or so and this will have a high impact on our academic community.

@cwestgarth
Copy link

Hi @hitteshahuja
We are hoping to get the fix out in the next couple of weeks, at the very latest.

Thanks

@cwestgarth
Copy link

Hi @hitteshahuja,

Apologies we currently do not have this feature available in the Plagiarism plugin yet. Please disregard the comment above.

Thanks

@hitteshahuja
Copy link

@cwestgarth So it does not affect the Plagiarism plugin then?

@cwestgarth
Copy link

Hi @hitteshahuja
No this won't affect the PP.

Thanks

@dwinn
Copy link
Contributor

dwinn commented Jul 26, 2016

This has now been fixed in the latest release v2016072601.

@ishanivardhan
Copy link

ishanivardhan commented Oct 10, 2019

Just needed to check if the feature to avoid sending notifications to tutors of different groups was ever implemented? Currently all users with Tutor role are getting assignment submission notification whether or not the student is part of their group. Is there a setting in place where we specify which tutor should get notifications OR tutors of which particular group should get notifications?

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

No branches or pull requests