Skip to content

fix: implement SendWelcomeEmailToUserAsync in ResendEmailService#20

Merged
Morgana-Claw merged 1 commit intodevelopfrom
fix/resend-email-service-missing-method
Apr 1, 2026
Merged

fix: implement SendWelcomeEmailToUserAsync in ResendEmailService#20
Morgana-Claw merged 1 commit intodevelopfrom
fix/resend-email-service-missing-method

Conversation

@Morgana-Claw
Copy link
Copy Markdown
Collaborator

Description

Implements the missing method in to comply with the interface contract.

Problem

The CI was failing with the following error:

  • CS0535: 'ResendEmailService' does not implement interface member 'IEmailService.SendWelcomeEmailToUserAsync(string, string)'

Solution

Added the method to following the same pattern used in and . This method sends a user registration welcome email via Resend SMTP without attachments.

Changes

  • Added implementation
  • Uses the same Resend SMTP configuration
  • Sends HTML formatted welcome email matching the style of other email service implementations
  • Includes proper logging and error handling

Related

@Morgana-Claw Morgana-Claw merged commit c3c4bdd into develop Apr 1, 2026
1 check passed
Morgana-Claw added a commit that referenced this pull request Apr 1, 2026
* feat: add welcome email for user registration

- Add SendWelcomeEmailToUserAsync method to IEmailService interface
- Implement method in SmtpEmailService and SendGridEmailService with branded HTML template
- Inject IEmailService into AuthApplication and call it fire-and-forget after successful registration
- Email includes: congratulations message, next steps, ClientManager branding
- No PDF attachment (unlike customer welcome email)
- Update AuthApplicationTests to include IEmailService mock

* fix: implement SendWelcomeEmailToUserAsync in ResendEmailService (#20)
@Morgana-Claw Morgana-Claw deleted the fix/resend-email-service-missing-method branch April 1, 2026 20:28
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.

1 participant