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

[REF] Update a few references to invoicing #21101

Merged
merged 1 commit into from
Aug 25, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

[REF] Update a few references to invoicing

Before

Legacy access to the invoicing setting

After

Civi::settings()->get('invoicing');

Technical Details

Comments

@civibot
Copy link

civibot bot commented Aug 12, 2021

(Standard links)

@civibot civibot bot added the master label Aug 12, 2021
@eileenmcnaughton
Copy link
Contributor Author

@mlutfy I can't quite decide whether to give up on these for now or push through & do them all - but here are a few more

@eileenmcnaughton
Copy link
Contributor Author

@mlutfy @seamuslee001 @demeritcowboy anyone up to look at this - I feel like it should be just a quick sanity check review

@demeritcowboy
Copy link
Contributor

Sure I can take a look.

@demeritcowboy
Copy link
Contributor

jenkins retest this please (since it's been a while)

@demeritcowboy demeritcowboy merged commit 7c8b016 into civicrm:master Aug 25, 2021
@eileenmcnaughton eileenmcnaughton deleted the inv3 branch August 25, 2021 19:01
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