Nikita weekly company summary email#3316
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Reviewed the code changes for the Total Org Summary Email feature. The form structure, field validation, and submission logic are clearly implemented, and the flow integrates well with the existing dashboard components. I was not able to test the end-to-end functionality due to the requirement of setting up a GCP account with billing details, but the frontend logic and UI implementation appear consistent with the intended behavior described in the PR. The component looks well-structured and aligns with the expected admin workflow. |
|
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Nikita,
I have tried to review your PR locally and did all the changes you have mentioned and even updated the REACT_FRONTEND_URL = http://localhost:5173 in backend env still have unable to receive an email. It keeps saying "Failed to send email." Do let me know if i am missing any step. I have added my email as recipient and have not touched any Puppeteer logic.




There was a problem hiding this comment.
Hi, I was able to receive the email. For anyone else reviewing this PR, please ensure you update your 'const uri' to the latest in the below-mentioned files:
db.js, addMembersToTeams.js, addNewField.js
On the other hand, the total org summary in general doesn't work so the attachment in the email is of a 'Something went wrong' and the 'Refresh Page' button.
The rest works as expected. Attached video below.
https://github.com/user-attachments/assets/071d6edb-25be-438b-a78a-aad27f205713
Also added myself as a recipient and tried sending an email from the frontend. That works too.

Overall, the main functionality works but needs a lot of new updates in the code, since it is pretty old.








Description
Related PRS (if any):
This frontend PR is related to the #1291backend PR.
…
Main changes explained:
…
How to test:
npm installandnpm ito run this PR locallyi. Verify if you are getting all the graphs in the email attachment
ii. if they are loaded (This depends on the performance of your system. Feel free to increase timeout value in reportsController. js
The timeout is set to 50 sec it should be adequate
Screenshots or videos of changes:
The email looks something like this
The attachment
Note:
I appreciate your patience 💖