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

5.5 #12763

Merged
merged 2 commits into from
Sep 4, 2018
Merged

5.5 #12763

merged 2 commits into from
Sep 4, 2018

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Contribution detail report fixes

Before

fatal errors under some circumstances

After

errors resolved

Technical Details

This is a backport of #12661 and #12660 in order to have the fixes in next release

Comments

@seamuslee001

@seamuslee001
Copy link
Contributor

adding merge on pass as i agree with the backport

@KarinG
Copy link
Contributor

KarinG commented Sep 3, 2018

Awesome

@seamuslee001
Copy link
Contributor

Jenkins re test this please

@eileenmcnaughton
Copy link
Contributor Author

@seamuslee001 test failed on server related issues. I'm going to merge this as the server issues seem to be a bit ongoing & the patches have been tested against master & will be tested again when the next full run happens

@eileenmcnaughton eileenmcnaughton merged commit 76c2160 into civicrm:5.5 Sep 4, 2018
@agh1 agh1 mentioned this pull request Sep 5, 2018
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.

5 participants