Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Andreas <[email protected]>
  • Loading branch information
MelissaAutumn and devmount authored May 23, 2024
1 parent 962c1b9 commit 7e1e754
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion backend/src/appointment/controller/mailer.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion backend/src/appointment/routes/schedule.py
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit 7e1e754

Please sign in to comment.