-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
Standardise fee_label in event workflow messages #27325
Conversation
We already fixed it to the token in the online template so make offline & test ones match civicrm@e90b263#diff-2652c5ed241e0c2a68dc79d97ce2b6056ad2bc8ce37aee9bce8dd69678a160a8R187
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
I gave this a try and am not getting anything coming through in any of the templates (with cache clearing). |
That includes the previous PR in 5.65, you can see in the PR after screenshots that the fee_level is missing, so we should probably backport whatever the fix is. |
Yeah - I thought event fee was part of ... event. But it isn't - it's participant.fee_label I think... |
Should work as event.fee_label, it's listed in CRM_Event_Tokens - but it doesn't. |
@larssandergreen yeah - but did that ever work or was it a mistake |
It's in the Event API |
It's just not in the list of exposed tokens, I'll put up a PR in a second. |
This should do it: #27372 |
Ok so ^^ is merged through now |
Yes, this one is good to merge now. |
We already fixed it to the token in
the online template so make offline & test ones match
e90b263#diff-2652c5ed241e0c2a68dc79d97ce2b6056ad2bc8ce37aee9bce8dd69678a160a8R187