Messenger - Target based on Attendance Status Issue

Hello Ross - Congratulations to you and the team for launch of v12. Keep up the good work.

However, am reporting an issue here - Messenger based on Attendance does not seem to work.

Steps to recreate:
Take attendance of a student - mark him absent for a given date say 25th June.
On 25th june itself, go to messenger and trigger a wall message with publishing date as 25th June, select attendance status as “Absent” and include Students and Parents.
Login with Student/Parent Account.
No Message arrives.

Please let us know your thoughts.

Hello, I am on the road for a few weeks, but have asked on of our developer team to look into this. Will let you know when we have a fix. Out if interest does the email/SMS part of the attendance target work? Thanks, Ross.

Let’s message is to be sent for “Absent” students on 25th June. Include Parents - Yes, Include Student - No.
Channel wise sending behavior,
1)SMS - only one of the Parents gets the SMS.
2)Email - only one of the Students gets the email
3)Wall Msg - Nothing happens (reported above)

Being able to send message on the basis of Attendance status is a very handy feature - something that many institutions look out for. Hope for a reliable solution (as is the case with rest of the features).

Some corrections - Here’s the updated comment. Please ignore the previous.

Let’s say message is to be sent for “Absent” students on 25th June. Include Parents - Yes, Include Student - No.
Channel wise sending behavior,
1)SMS - only one of the Parents gets the SMS.
2)Email - only one of the Parents gets the email
3)Wall Msg - Nothing happens (reported above)

Being able to send message on the basis of Attendance status is a very handy feature - something that many institutions look out for. Hope for a reliable solution (as is the case with rest of the features).

BTW - have a safe and enjoyable trip, Ross.

Hi there. I’ve looked at the code and found what was causing the wall messages to not appear. I’ve also found what was causing only one Email to send and fixed that as well, however I don’t have an SMS system setup to test (although based on the code changed the fix for email should also apply to SMS sending).

I’ve created a commit for it which is compatible with v12. The fix will be added to the core for v13 after the summer when Ross is back from holidays, for now you’re welcome to update your install manually.

There are only two files to change so it should be relatively quick to update by hand:

Replace your modules/Messenger/moduleFunctions.php with this one:
https://github.com/SKuipers/core/blob/feature/Messenger_Absentee/modules/Messenger/moduleFunctions.php

Replace your modules/Messenger/messenger_postProcess.php with this one:
https://github.com/SKuipers/core/blob/feature/Messenger_Absentee/modules/Messenger/messenger_postProcess.php

You can see a summary of the exact changes made in the following pull request:
https://github.com/GibbonEdu/core/pull/80/files

Give a shout if you have any questions or further messages not sending.

Thanks!

Amazing Sandra! Will look at that pull request when I return from holiday : )

Include Parents ONLY - while messaging

For Today’s Attendance - Wall Message/Email/SMS - Works Great!

For Past Dated Attendance - Email/SMS - Works Great!
For Past Dated Attendance - Wall Message - Nothing is sent; Please revisit

Include Students ONLY - while messaging

For Today’s Attendance - Wall Message- Works Great!
For Today’s Attendance - Email - Only 1 email sent; Please revisit
For Today’s Attendance - SMS - Only 1 sms sent; Please revisit

For Past Dated Attendance - Wall Message- Nothing is sent; Please revisit!
For Past Dated Attendance - Email- Only 1 email sent; Please revisit.
For Past Dated Attendance - SMS- Only 1 sms sent; Please revisit!

I have not yet tested combination messaging (Including both Parents and Students with multiple Attendance Status).

@ross - must mention that am AMAZED by your lightening response - resolving the issue at large; Few more revisits your way. All the best.

@skuipers Do you want me to retest or need any further explanation?