Anthony/Warning-Tracker-Followup#4502
Merged
one-community merged 9 commits intodevelopmentfrom Dec 16, 2025
Merged
Conversation
…d bluesquare email assignment, and implement issue blue square action through warning trackers
…on restriction works, and modified toast messages
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
…n code % for sonarqubecloud bot
Anusha-Gali
previously approved these changes
Dec 12, 2025
|
Member
|
Thank you all, merging! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

















Description
Follow up to PR#3978, adding the feature to add a blue square to a user, through choosing to issue a blue square through the warning trackers, both on the dashboard and on the user profile.
Related PRS (if any):
This PR is related to PR#1938 backend PR.
…
Main changes explained:
…
How to test:
check into current branch
do npm install and ... to run this PR locally
Clear site data/cache
log as admin user (owner needed to test being able to add or remove the Blue Square Email Management permission though)
Issue Warnings On Dashboard
Find your account or an account you own, that you'll be testing with
Click on the Tracking Button (yellow button that should change to show Hide, and display the warnings list below it)
Click on the circles of any warning, to simulate adding a warning of that kind for the user
Check if you get a success toast with this message: 'Successfully logged and tracked'
Repeat for same warning so that when attempting to add a 3rd warning (after having added 2 blue circles), you'd get the modal pop up asking if you'd like to issue another warning or a blue square
Issue a warning and verify you get this success toast message: 'Warning successfully logged and sent by email.'
Repeat step 10 but issue a bluesquare and verify you get the success toast message: 'Successfully logged and Blue Square issued on profile and by email.' if nothing goes wrong.
User Profile Blue Square Check, Issue Special Warnings, and Test Both Button Blue Squares and Mixed Cases
Go to a user profile (preferably yours or another account you made that you can access the email for)
Check the blue squares and locate added blue square from the dashboard steps, should look like what's shown in the image. (Note: Only difference between blue squares issued from dashboard will be the name of the warning tracker, indicated by the ""s)
Next, click any blue square and remove it by clicking the RBS4NS (Remove Blue Square for No Summary) yellow button, and click Issue Blue Square
Verify it removes the bluesquare you chose, then adds a new bluesquare with text explaining it. (This will be different than the blue squares issued from dashboard, so don't mind it)
Test with repeating the same step but with the RBS4HCE (Remove Blue Square for Hours Close Enough), ensure the text has the right warning name mentioned
Test using the Both, and verify if the summary text accurately describes what you chose. (Ex. If you chose to issue blue square for both warnings, then the summary text should say blue square for both. If you chose blue square for no summary, and warning for hours close enough, then the text will say blue square for no summary, and a warning for hours close enough.
Also, verify if the number of times listed in the summary text, matches the Occurrence number of the respective warning
Check Blue Square Email Management permission (Owner users only)
Go to Permissions Management
Click on a role, or Manage User Permissions and select a user
Scroll down to Misc/Unsorted, and find Blue Square Email Management
Verify if you can click on it, and change it between Add/Delete
Note
If you have setup email functionality for your local end by following Email sending instructions, then if you issue blue squares or warnings, you will have to modify emailSender function calls, otherwise you may end up emailing other people than just yourself. So adjust them to only be your own emails before testing those features directly. Or you could just set sendEmail to false in the backend env file, as I think that would ensure no email gets sent either.
If you haven't setup your environment for email functionality, then you shouldn't have to worry about accidentally sending emails to people, but double check though incase I forgot something.