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

Applies #17 for non-redirect processors. #73

Closed
wants to merge 1 commit into from

Conversation

mlutfy
Copy link
Contributor

@mlutfy mlutfy commented Feb 5, 2019

I ran into #17, but with Beanstream, which is a non-redirect API.

@civibot civibot bot added the master label Feb 5, 2019
@eileenmcnaughton
Copy link
Owner

Hmm I wonder if I can add a serialize function to the class - to get away from this http://php.net/manual/en/class.serializable.php

eileenmcnaughton added a commit that referenced this pull request Mar 4, 2019
#73 attempt more generic fix on serialization
@eileenmcnaughton
Copy link
Owner

I think we otherwise fixed this by adding a serialize function to the class

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.

None yet

2 participants