Skip to content

Comments

v1.4.0#18

Merged
andrebolina merged 2 commits intomainfrom
feat/email-integration
Feb 23, 2026
Merged

v1.4.0#18
andrebolina merged 2 commits intomainfrom
feat/email-integration

Conversation

@andrebolina
Copy link
Member

@andrebolina andrebolina commented Feb 23, 2026

Summary

This pull request introduces email notification functionality for key user actions and adds support for storing user email addresses. The main changes include integrating the Resend email service, sending notifications when new accounts or reviews are created, and updating the account and review creation flows to support these features. Additionally, the GitHub OAuth scope and user data fetching have been enhanced to retrieve email addresses.

Email notification integration:

  • Added EmailService class to handle sending notifications for new accounts and reviews via Resend, including configuration checks and email formatting.
  • Introduced new environment variables (RESEND_API_KEY, SENDER_EMAIL, ABUSE_CONTROL_EMAIL) for email service configuration in .env.example and settings.

Account and review creation enhancements:

  • Updated account creation flow to fetch and store user email from GitHub, and send a notification when a new account is created.
  • Modified review creation/updating flow to return a tuple indicating if a review is new, and send a notification email for new reviews.

Type of change

  • New feature
  • Bug fix
  • Minor change

Validation

  • Backend lint: cd backend && uv run ruff check app tests
  • Backend type check: cd backend && uv run mypy
  • Backend tests: cd backend && uv run pytest
  • Frontend lint: cd frontend && npm run lint
  • Frontend tests: cd frontend && npm run test:run

Checklist

  • I did not include secrets or private data.
  • I added/updated tests for behavior changes.
  • I updated version in VERSION, CHANGELOG.md, and README.md.
  • I updated docs where needed.

@andrebolina andrebolina self-assigned this Feb 23, 2026
@andrebolina andrebolina merged commit 8e00e1a into main Feb 23, 2026
8 checks passed
@andrebolina andrebolina deleted the feat/email-integration branch February 23, 2026 01:58
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