Skip to content

Wire risk alert email after document analysis#860

Open
bingmokaka wants to merge 1 commit into
CodeGirlsInc:mainfrom
bingmokaka:wire-risk-alert-email
Open

Wire risk alert email after document analysis#860
bingmokaka wants to merge 1 commit into
CodeGirlsInc:mainfrom
bingmokaka:wire-risk-alert-email

Conversation

@bingmokaka

Copy link
Copy Markdown

Summary

  • inject MailService and UsersService into the analyze job processor
  • send risk alert emails only when document analysis returns one or more flags
  • catch and log email delivery failures so analysis jobs still complete

Verification

  • nest build passes after temporarily exposing the existing transitive dotenv package under pnpm strict resolution
  • note: the repo manifest does not declare dotenv directly even though src/main.ts imports it, so plain pnpm build is blocked by that pre-existing dependency declaration issue

Closes #781

@vercel

vercel Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

@bingmokaka is attempting to deploy a commit to the Mftee's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

[BE-57] Wire MailService.sendRiskAlert() into DocumentProcessor after analyze job completes

1 participant