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

Pass template_type through to alterMailing hook #16529

Merged
merged 1 commit into from
Feb 19, 2020

Conversation

mattwire
Copy link
Contributor

Overview

Follow on from #15306. Knowing the type of template (ie. traditional or mosaico) allows us to behave differently depending on the type of template.

Before

template_type available but not passed to hook.

After

template_type passed to hook.

Technical Details

Comments

@bhahumanists Please can you create a docs PR for this?

@civibot
Copy link

civibot bot commented Feb 13, 2020

(Standard links)

@eileenmcnaughton
Copy link
Contributor

@mattwire this seems fine - it's not a new hook or a significant change to the existing one - once you've addressed the needs-documentation go ahead & merge it.

@bhahumanists
Copy link
Contributor

@mattwire done

@eileenmcnaughton eileenmcnaughton merged commit 611272e into civicrm:master Feb 19, 2020
@mattwire mattwire deleted the altermailing_templatetype branch February 19, 2020 20:28
@homotechsual
Copy link
Contributor

Clear "needs-documentation".

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

Successfully merging this pull request may close these issues.

4 participants