Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add initial Event Message Template previewability #24323

Merged
merged 1 commit into from Aug 25, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Aug 20, 2022

Overview

Add initial Event Message Template previewability.

Note this PR also makes participant and event tokens work in the event_online template - they were only working in the offline before this

Before

No event template preview possible

After

Initial functionality added - event tokens are an easy add here to I fixed some very simple variables to be tokens to get this far

image

Technical Details

This adds if for online & offline receipt. The other participant templates will be a follow on once this is merged

Comments

Pre-existing test cover in https://github.com/civicrm/civicrm-core/pull/24323/files#diff-c4ddaa8aa71ca49d31466c04cefbb2f1d608a04e3836791a756859d73ef560c2

@demeritcowboy hopefully this is an easy-ish merge as it is similar to other recent ones

@civibot
Copy link

civibot bot commented Aug 20, 2022

(Standard links)

@civibot civibot bot added the master label Aug 20, 2022
@demeritcowboy
Copy link
Contributor

Is it expected that there's no fee information in the preview? This is just to get it working?

Otherwise this looks ok just some notices that I think were there before too.

@eileenmcnaughton
Copy link
Contributor Author

@demeritcowboy yep - that's expected - it's just a very limited initial add

@demeritcowboy
Copy link
Contributor

okey dokey

@demeritcowboy demeritcowboy merged commit 5a66dce into civicrm:master Aug 25, 2022
@seamuslee001 seamuslee001 deleted the balance branch August 25, 2022 01:22
@eileenmcnaughton
Copy link
Contributor Author

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants