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

Fix parameter format for upgrade call to install/enable sequentialcreditnotes #16686

Merged
merged 1 commit into from
Mar 4, 2020
Merged

Fix parameter format for upgrade call to install/enable sequentialcreditnotes #16686

merged 1 commit into from
Mar 4, 2020

Conversation

demeritcowboy
Copy link
Contributor

Overview

The third parameter doesn't have the expected format, so it silently doesn't do anything.

@civibot civibot bot added the master label Mar 4, 2020
@civibot
Copy link

civibot bot commented Mar 4, 2020

(Standard links)

@demeritcowboy demeritcowboy force-pushed the install-seqcreditnotes branch from 9a20881 to cdcba4b Compare March 4, 2020 21:35
@totten
Copy link
Member

totten commented Mar 4, 2020

This looks pretty sensible though I haven't tested. It'd be good to get this merged before 5.24 RC branches.

@eileenmcnaughton eileenmcnaughton merged commit 70c0fed into civicrm:master Mar 4, 2020
@demeritcowboy demeritcowboy deleted the install-seqcreditnotes branch March 5, 2020 01:21
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
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@demeritcowboy @totten @eileenmcnaughton