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.

Sign up for GitHub

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

Jump to bottom

[NFC] ENsure that formTpl is assigned in CRM_Custom_Form_OptionTest #22065

Merged
merged 1 commit into from
Nov 12, 2021
Merged

[NFC] ENsure that formTpl is assigned in CRM_Custom_Form_OptionTest #22065

merged 1 commit into from
Nov 12, 2021

Conversation

seamuslee001
Copy link
Contributor

Overview

Like #22061 but for the OptionTest

Before

Test fails in php8

After

test passes

ping @eileenmcnaughton @colemanw @demeritcowboy

@civibot civibot bot added the master label Nov 12, 2021
@civibot
Copy link

civibot bot commented Nov 12, 2021

(Standard links)

@seamuslee001 seamuslee001 mentioned this pull request Nov 12, 2021
Merged
@seamuslee001 seamuslee001 merged commit 233461b into civicrm:master Nov 12, 2021
@demeritcowboy
Copy link
Contributor

Take that php and smarty. 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master merge on pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@seamuslee001 @demeritcowboy