To limit abuse, our points system only allows gives points for a maximum of 5 mentions in a single message. Currently, the "you can only do 5 at a time..." message is sent even when only 5 mentions are made, which seems out of place. Ideally, it's only sent if 6+ mentions are included, so people know only 5 of them actually received points.
This PR
Amends points.js to only send the 5-limit message if more than 5 mentions were included.
Amends tests for this limit so they actually test when exceeding 5 points mentions in a single message. No snapshots needed to be updated.
Because
To limit abuse, our points system only allows gives points for a maximum of 5 mentions in a single message. Currently, the "you can only do 5 at a time..." message is sent even when only 5 mentions are made, which seems out of place. Ideally, it's only sent if 6+ mentions are included, so people know only 5 of them actually received points.
This PR
points.jsto only send the 5-limit message if more than 5 mentions were included.points.jsusing our Prettier rule set.Issue
N/A
Additional Information
Pull Request Requirements
location of change: brief description of changeformat, e.g.Callbacks command: Update verbiageBecausesection summarizes the reason for this PRThis PRsection has a bullet point list describing the changes in this PRIssuesection