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

CRM-20588, Add Pre and Post hook for Batch #10364

Merged
merged 3 commits into from
May 22, 2017
Merged

CRM-20588, Add Pre and Post hook for Batch #10364

merged 3 commits into from
May 22, 2017

Conversation

pradpnayak
Copy link
Contributor

@pradpnayak pradpnayak commented May 16, 2017 •
edited by civicrm-builder
Loading

pradpnayak added 2 commits May 17, 2017 03:55
@pradpnayak
CRM-20588, code cleanup
92e088c 
----------------------------------------
* CRM-20588: Pre and Post hook for Batch
  https://issues.civicrm.org/jira/browse/CRM-20588
@pradpnayak
CRM-20588, added pre and post hook invocation
debff43 
----------------------------------------
* CRM-20588: Pre and Post hook for Batch
  https://issues.civicrm.org/jira/browse/CRM-20588
@pradpnayak
Copy link
Contributor Author

This PR contains

  1. code cleanup i.e removed extra parameters used in Batch create() function and made necessary changes in other files.
  2. Add pre and post hook invocation for batch

@pradpnayak
CRM-20588, add pre and post hook invocation for delete option
8f30d0b 
----------------------------------------
* CRM-20588: Pre and Post hook for Batch
  https://issues.civicrm.org/jira/browse/CRM-20588
@pradpnayak
Copy link
Contributor Author

@eileenmcnaughton @colemanw could you please review this?

@colemanw colemanw merged commit a39f4b3 into civicrm:master May 22, 2017
@monishdeb monishdeb deleted the CRM-20588 branch May 22, 2017 15:07
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
@pradpnayak @colemanw @civicrm-builder