From 7e1e7541617d3ff5d3c0d9f6d208b71b4db56ef4 Mon Sep 17 00:00:00 2001 From: Mel <97147377+MelissaAutumn@users.noreply.github.com> Date: Thu, 23 May 2024 08:07:23 -0700 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Andreas --- backend/src/appointment/controller/mailer.py | 2 +- backend/src/appointment/routes/schedule.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/backend/src/appointment/controller/mailer.py b/backend/src/appointment/controller/mailer.py index 09d34d657..12798bfa3 100644 --- a/backend/src/appointment/controller/mailer.py +++ b/backend/src/appointment/controller/mailer.py @@ -170,7 +170,7 @@ def text(self): class ConfirmationMail(Mailer): def __init__(self, confirm_url, deny_url, attendee_name, attendee_email, date, *args, **kwargs): """init Mailer with confirmation specific defaults""" - self.attendee_name = attendee_name.title() + self.attendee_name = attendee_name self.attendee_email = attendee_email self.date = date self.confirmUrl = confirm_url diff --git a/backend/src/appointment/routes/schedule.py b/backend/src/appointment/routes/schedule.py index caffc274c..05b89c741 100644 --- a/backend/src/appointment/routes/schedule.py +++ b/backend/src/appointment/routes/schedule.py @@ -254,7 +254,7 @@ def request_schedule_availability_slot( db.commit() # Sending confirmation email to owner - background_tasks.add_task(send_confirmation_email, url=url, attendee_name=attendee, date=date, to=subscriber.email) + background_tasks.add_task(send_confirmation_email, url=url, attendee_name=attendee.name, date=date, to=subscriber.email) # Sending pending email to attendee background_tasks.add_task(send_pending_email, owner_name=subscriber.name, date=attendee_date, to=slot.attendee.email)