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

Remove extraneous debug #20319

Merged
merged 1 commit into from
May 19, 2021
Merged

Remove extraneous debug #20319

merged 1 commit into from
May 19, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Remove extraneous debug

Before

blah blah

After

poof

Technical Details

We added this a while back when we were getting reports of an error that
we couldn't track down. We never got feedback of these lines appearing in the
logs and the error appears to be otherwise solved so
I think the debug adds more confusion & has not proven helpful

Comments

We added this a while back when we were getting reports of an error that
we couldn't track down. We never got feedback of these lines appearing in the
logs and the error appears to be otherwise solved so
I think the debug adds more confusion & has not proven helpful
@civibot
Copy link

civibot bot commented May 17, 2021

(Standard links)

@civibot civibot bot added the master label May 17, 2021
@eileenmcnaughton
Copy link
Contributor Author

@colemanw @seamuslee001 is one of you comfortable merging this? It's debug I added but it never seemed to help us find the issue so I think it's just confusing now

@seamuslee001
Copy link
Contributor

Yeh i'm ok with removing this

@seamuslee001 seamuslee001 merged commit 245d256 into civicrm:master May 19, 2021
@seamuslee001 seamuslee001 deleted the pp branch May 19, 2021 21:43
@eileenmcnaughton
Copy link
Contributor Author

thanks @seamuslee001

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants