alfio-event / alf.io

alf.io - The open source ticket reservation system for conferences, trade shows, workshops, meetups
https://alf.io
GNU General Public License v3.0
1.38k stars 345 forks source link

Custom email email token #1269

Open tobsowo opened 1 year ago

tobsowo commented 1 year ago

Sending an email using the {{eventName}} token showed correctly during the preview but when the email was sent, it used the slug in the event.

cbellone commented 1 year ago

Note for first-time contributors: this issue is referring to the "compose custom message" functionality. To access it:

image

ved-asole commented 1 year ago

Thanks @cbellone, I will go through the process and would try to take care of the above bug

ved-asole commented 1 year ago

Sending an email using the {{eventName}} token showed correctly during the preview but when the email was sent, it used the slug in the event.

@tobsowo - If possible can you share a sample of what inputs you gave and what kind of output email was sent along with the slug?

cbellone commented 1 year ago

Thanks @cbellone, I will go through the process and would try to take care of the above bug

Please note that this fix must be done on the 2.0-M4-maintenance branch

ved-asole commented 1 year ago

sure

tobsowo commented 1 year ago

@ved-asole I'm not on PC, I'll do that once I get the chance.

ved-asole commented 1 year ago

sure, Thanks @tobsowo 😊

tobsowo commented 1 year ago

If you look at the token on the right {{eventName}} is interpreted as sep23 but in the preview it was the right token. When the email is sent, it is sep23 that is used instead of the event name. image

Preview image

ved-asole commented 12 months ago

Oh. ok @tobsowo. So you mean that the {{eventName}} value should be the Unlocking Marketing Power: Integrating Mautic and Wordpress [Meetup] as per your configuration but it's coming as sep23 ? Please correct me if I'm wrong.

tobsowo commented 12 months ago

You're correct.

ved-asole commented 12 months ago

Thanks for confirming. I will check and resolve it.

ved-asole commented 11 months ago

Hi @cbellone, thanks for approving the pull request.

Hi @tobsowo, this issue is resolved as part of the 2.0-M4-maintenance branch. It will deployed in 2.0-M4-maintenance version.

tobsowo commented 11 months ago

Thank you for the update