Limit list for forum recipients to 6 month since last login
Touches #64 (closed)
What does this MR do?
Limit mail recipients for forum notifications to users that have logged in during the last 6 month.
How confident are you it won't break things if deployed?
Quite
Links to related issues
How to test
Steps a reviewer can take to verify that this MR does what it says it does e.g.
- Checkout branch locally
- Login as foodsaver
- Go to http://localhost:18084
- Create a new forum post in Göttingen
- See how many mails would be sent. 6a. different ways for the next step , one is to // the linke with the AND last_login part 6a. create forum post again 6a. now it should show more mails 6b. Use the select against the DB to see what the differenc in number is for users since their last login.
Screenshots (if applicable)
Any relevant screenshots if this is a design / frontend change
Checklist
-
added a test, or explain why one is not needed/possible... -
no unrelated changes -
asked someone for a code review -
joined #foodsharing-beta channel at https://slackin.yunity.org -
added an entry to CHANGELOG.md (description, merge request link, username(s)) -
Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Johannes