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

Grade emails aren't sent consistently #322

Open
lieuwex opened this issue Mar 16, 2017 · 6 comments
Open

Grade emails aren't sent consistently #322

lieuwex opened this issue Mar 16, 2017 · 6 comments
Labels

Comments

@lieuwex
Copy link
Member

lieuwex commented Mar 16, 2017

As reported by a friend, he said that a new grade B7de6PTs8yDpWAMma was notified by mail, but grade 2zz6dtDpjSsdruAmR wasn't.

@lieuwex lieuwex added the bug label Mar 16, 2017
@lieuwex
Copy link
Member Author

lieuwex commented Mar 16, 2017

It seems it hasn't been sent at all, so it's not a bounce or smth at sparkpost

@ghost
Copy link

ghost commented Mar 16, 2017

So, it has to be something with the eventhandler. Probably

@lieuwex
Copy link
Member Author

lieuwex commented Mar 16, 2017

@marcus-talbot We don't use an event handler, we check every 15 minutes using a cron job: https://github.com/simplyGits/simplyHomework/blob/develop/packages/notices/server/emails.js#L32-L107

@ghost
Copy link

ghost commented Mar 16, 2017

Didn't look at it properly yet, but maybe it notices new grades, sends a mail for the first new grade id, and then just returns, skipping the rest of the grades all together?

@tomsmeding
Copy link
Member

I see that you filter on grades that are newly entered in the past 15 minutes in a repeating job that runs every 15 minutes. How accurate is the syncedcron timer? Maybe it's possible that a grade is entered RIGHT after your syncedcron job and syncedcron is a bit slow in launching the next one, so that the grade is too late for the previous run, but just too early for the next.

@lieuwex
Copy link
Member Author

lieuwex commented Mar 16, 2017

@marcus-talbot could be, I think I have tested it though but I can take a look

@tomsmeding Yeah I was thinking of something like that too, I have to dive into the logs to see that, I will when I've time.

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

No branches or pull requests

2 participants