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

#2892 Replace event type token in action schedule changes event.event_type_id to participant.event_type_id:label. But should be event.event_type_id to event.event_type_id:label #21738

Merged
merged 1 commit into from
Oct 6, 2021

Conversation

agileware-justin
Copy link
Contributor

Replace event type token in action schedule changes event.event_type_id to participant.event_type_id:label. But should be event.event_type_id to event.event_type_id:label

As identified by @magnolia61 here https://github.com/civicrm/civicrm-core/pull/21666/files#r720059285

Related https://lab.civicrm.org/dev/core/-/issues/2892

…t.event_type_id to participant.event_type_id:label. But should be event.event_type_id to event.event_type_id:label
@civibot
Copy link

civibot bot commented Oct 6, 2021

(Standard links)

@eileenmcnaughton
Copy link
Contributor

thanks @agileware-justin !!

@eileenmcnaughton
Copy link
Contributor

As an aside - this PR #21706 makes it so we can have tokens work without being 'advertised' in the widget & suppresses the ones normal users wouldn't use from the widget

@agileware-justin
Copy link
Contributor Author

Ta have subscribed to #21706

@eileenmcnaughton
Copy link
Contributor

@agileware-justin if you are doing other testing maybe you could pull in #21706 & include it - I think a 'yes this works' would be helpful in getting it merged

@eileenmcnaughton eileenmcnaughton merged commit 5fc1244 into civicrm:master Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants