Skip to content

fix: resolve TypeScript build error in notifyMail SMTP transporter#63

Merged
Atanu2k4 merged 2 commits into
sanks011:masterfrom
Atanu2k4:master
Jun 1, 2026
Merged

fix: resolve TypeScript build error in notifyMail SMTP transporter#63
Atanu2k4 merged 2 commits into
sanks011:masterfrom
Atanu2k4:master

Conversation

@Atanu2k4

@Atanu2k4 Atanu2k4 commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes a TypeScript compilation error introduced when adding Railway-specific SMTP options
(family, connectionTimeout, etc.) to the Nodemailer transporter. The family: 4
property is not present in Nodemailer's default TransportOptions type, causing the build
to fail with a "No overload matches this call" error.

Related issue

Fixes #62

Changes

  • Added import type SMTPTransport from "nodemailer/lib/smtp-transport" in lib/notifyMail.ts
  • Cast the createTransport(...) options object as SMTPTransport.Options to satisfy TypeScript
    while preserving all runtime SMTP settings (family: 4, timeouts, TLS config)

Testing

  • npm run build completes successfully with no TypeScript errors
  • All 31 routes compile and generate correctly

Checklist

  • Title follows the repository PR format
  • The issue is linked with Fixes #62
  • Testing notes are included

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

PR Validation Passed

Your PR title and description meet all requirements:

@Atanu2k4 Atanu2k4 merged commit 836c5a0 into sanks011:master Jun 1, 2026
1 check passed
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.

BUG: smtp not working

1 participant