Linh - Fix edit/save functionality in Weekly Summary Email for Admins#3927
Linh - Fix edit/save functionality in Weekly Summary Email for Admins#3927linh2020 wants to merge 3 commits intodevelopmentfrom
Conversation
…stants to resolve task issues
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Naveen-Kumar-Reddy-27
left a comment
There was a problem hiding this comment.
Navigated to Weekly Summary → clicked Weekly Summary Email.
Verified editing, saving, adding, and deleting recipient emails → all changes persisted correctly.
Tested in both light and dark mode → functionality works fine.
Observation:
In dark mode, the text “Set Weekly Summary Email Recipients” is hard to read due to low contrast.
Everything else is working as expected.

There was a problem hiding this comment.
Hi Linh,
I have reviewed your PR locally and though the functionality work in light mode however in dark mode, the whole page is still in light mode and the main functionality though is well adjusted to dark mode, the title isn't. Also on a side note: is the backend working cause i do not see my name in the db.
…ighestGoodNetworkApp into linh_fix_weekly_summary_email_update
7377c5e to
bef8409
Compare
|
















…stants to resolve task issues
Description
This PR addresses issues reported in the Weekly Company Summary Email for Admins feature, originally introduced in PR #3671
The main issue was that edited email addresses were not being updated immediately after saving changes. This PR fixes that behavior and ensures that changes are reflected in the recipient list without requiring a manual refresh.
Related PRS (if any):
This frontend PR is related to the backend PR OneCommunityGlobal/HGNRest#1478
Main changes explained:
How to test:
Confirm the updated email appears instantly in the list.
Screenshots or videos of changes:
Note:
This PR fixes the edit + save bug identified in the last developer’s PR.
All tests (npm test) and lint checks pass locally.