Feature/mail#95
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Caution Review failedThe pull request is closed. WalkthroughEmail notification system integrated across authentication, appointment, and prescription workflows. Server-side controllers now send confirmation and status emails via refactored emailService. Email configuration updated to parse environment variables. Minor UI styling adjustments applied to dashboard components. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant AuthController
participant EmailService
participant Email Provider
Client->>AuthController: POST /register (email, password, role)
AuthController->>AuthController: Validate & create user
AuthController->>AuthController: Generate tokens
activate AuthController
AuthController->>EmailService: sendWelcomeEmail(email, userName, role)
EmailService->>Email Provider: Send welcome email
Email Provider-->>EmailService: Sent (async, non-blocking)
EmailService-->>AuthController: Returns info
deactivate AuthController
AuthController-->>Client: Return tokens & user data
sequenceDiagram
participant Client
participant AppointmentController
participant EmailService
participant Email Provider
Client->>AppointmentController: POST /book (appointmentData)
AppointmentController->>AppointmentController: Create appointment
AppointmentController->>AppointmentController: Populate user/doctor/clinic data
activate AppointmentController
AppointmentController->>EmailService: sendAppointmentBookedEmail(patientEmail, appointmentDetails)
EmailService->>Email Provider: Send confirmation email
Email Provider-->>EmailService: Sent (async)
EmailService-->>AppointmentController: Returns info
deactivate AppointmentController
AppointmentController-->>Client: Return appointment
sequenceDiagram
participant Client
participant AppointmentController
participant EmailService
participant Email Provider
Client->>AppointmentController: PUT /update (appointmentData, status)
AppointmentController->>AppointmentController: Update appointment with status
AppointmentController->>AppointmentController: Populate updated data
alt Status provided
activate AppointmentController
AppointmentController->>EmailService: sendAppointmentStatusEmail(patientEmail, appointmentDetails, newStatus)
EmailService->>Email Provider: Send status change email
Email Provider-->>EmailService: Sent (async)
EmailService-->>AppointmentController: Returns info
deactivate AppointmentController
end
AppointmentController-->>Client: Return updated appointment
sequenceDiagram
participant Client
participant PrescriptionController
participant EmailService
participant Email Provider
Client->>PrescriptionController: POST /create (prescriptionData)
PrescriptionController->>PrescriptionController: Create & link prescription
PrescriptionController->>PrescriptionController: Fetch populated prescription with patient email
activate PrescriptionController
PrescriptionController->>EmailService: sendPrescriptionEmail(patientEmail, prescriptionDetails)
EmailService->>Email Provider: Send prescription email
Email Provider-->>EmailService: Sent (async, non-blocking on errors)
EmailService-->>PrescriptionController: Returns info
deactivate PrescriptionController
PrescriptionController-->>Client: Return prescription
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Possibly related PRs
Suggested labels
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (9)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
📋 PR Auto-Check📝 Files Changed✨ Code QualityPrettier Check: ✅ Passed Auto-generated on 2025-11-15T10:50:23.100Z |
📋 PR Auto-Check📝 Files Changed✨ Code QualityPrettier Check: ✅ Passed Auto-generated on 2025-11-15T10:52:17.500Z |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Project
Change Type
Stack
Page Type
Route/API Endpoint Status
What Changed
Route/API Affected
Description
Screenshots (If Applicable)
Code Quality
npx prettier --check .)Related Issues
Closes #94
Summary by CodeRabbit
New Features
Style
Chores