Skip to content

Conversation

@guillermobermejo
Copy link
Collaborator

@guillermobermejo guillermobermejo commented Nov 21, 2025

Description

This PR introduces a working email-sending workflow for the Wallet app by adding a new sendEmail helper inside packages/common along with a full end-to-end (E2E) test using a local MailHog SMTP server.

The goal is to safely validate outgoing email functionality without sending anything to real user inboxes. This prepares the project for future integration with the organization’s Gmail/Workspace SMTP configuration.

Fixes: #632
Resolves: N/A


Changes Made

  • Changes in apps folder
    No app changes included.

  • Changes in packages folder:

    • packages/common
      • Added new sendEmail.ts helper using Nodemailer (createTransport(...))
      • Added packages/common/spec/e2e/sendEmail.spec.ts to verify full email delivery
      • Added E2E Jest config (jest.e2e.config.cjs, tsconfig.jest.json)
      • Updated root package.json with test:e2e script

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 💥 Breaking change
  • 📝 Documentation update

How Has This Been Tested?

The email workflow was validated using a local MailHog SMTP server running inside Docker.
This ensures all outgoing emails are captured locally and nothing is sent externally.

1. Start MailHog locally (Docker)

docker run -d --name mailhog -p 1025:1025 -p 8025:8025 mailhog/mailhog
docker ps -a
docker start mailhog
yarn test:e2e

@guillermobermejo
Copy link
Collaborator Author

Sorry for the delay following Tuesday’s meeting. I had to fix a few issues after fetching upstream and rebasing — mainly conflicts in package.json. Everything is now resolved, and I was able to get sendEmail.ts working end-to-end using Nodemailer again, delivering successfully to a local fake SMTP server (MailHog).

@dadiorchen
Copy link
Collaborator

@guillermobermejo do you get the gmail setting? So you can use that to finish the e2e test?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create sending email e2e test

2 participants