UoaWDCC / ASPA-EnrollmentForm

6 stars 1 forks source link

[ASPA-8] Add name display to confirmation email and change event photo #95

Closed Metarock closed 3 years ago

Metarock commented 3 years ago

Issue: Currently when a confirmation email is sent to a recipient, "Hi there" is displayed, it should be "Hi {personName}". Additionally, background photo is not generic.

Solution: Added another argument into the function sendConfirmationEmail which takes in the recipient's name. public function sendConfirmationEmail(string $recipientName, string $recipientEmail, string $paymentMethod, array $eventData)

Also changed the line 280 to the correct name self::sanitize($recipientName)

To change the background photo, change this line's URL to the desired background photo $EVENT_IMAGE = "https://user-images.githubusercontent.com/19633284/115980245-417e0500-a5df-11eb-9741-3b7a10499ef5.png";

Risk: HTML injection

Reviewed By: [Edit the message as to who reviewed it here]