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

[REF] Move another function to the FinancialProcessor #21914

Merged
merged 1 commit into from
Oct 27, 2021
Merged

[REF] Move another function to the FinancialProcessor #21914

merged 1 commit into from
Oct 27, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

[REF] Move another function to the FinancialProcessor

Before

Function on contribution BAO

After

Moved to FInancialProcessor - where we are consolidating all the functions for financial processing - in preparation for cleaning them up

Technical Details

At this stage just moving all the financial processing functions to the same class

Comments

@civibot
Copy link

civibot bot commented Oct 23, 2021

(Standard links)

@civibot civibot bot added the master label Oct 23, 2021
@eileenmcnaughton
[REF] Move another function to the FinancialProcessor
051549d 
At this stage just moving all the financial processing functions to the same class
@eileenmcnaughton
Copy link
Contributor Author

test this please

@demeritcowboy
Copy link
Contributor

jenkins test this please

@demeritcowboy demeritcowboy merged commit e236f74 into civicrm:master Oct 27, 2021
@eileenmcnaughton eileenmcnaughton deleted the trxn branch October 27, 2021 19:47
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
@eileenmcnaughton @demeritcowboy