Skip to content

Errors array#236

Open
teolaz wants to merge 5 commits into
drewm:masterfrom
teolaz:errors-array
Open

Errors array#236
teolaz wants to merge 5 commits into
drewm:masterfrom
teolaz:errors-array

Conversation

@teolaz

@teolaz teolaz commented Mar 19, 2018

Copy link
Copy Markdown

After the request here #235 , i updated the Mailchimp class and Readme to add the getErrorsArray method.
Please tag the release to 2.6

@drewm

drewm commented Mar 19, 2018

Copy link
Copy Markdown
Owner

This seems to roll back #231. Was there a reason for that, or was it just a mistake?

@drewm

drewm commented Mar 19, 2018

Copy link
Copy Markdown
Owner

I think the errors array should be populated within determineSuccess() as that is where last_error is set.

@teolaz

teolaz commented Mar 19, 2018

Copy link
Copy Markdown
Author

Populated as your request.

@teolaz

teolaz commented Mar 19, 2018

Copy link
Copy Markdown
Author

This seems to roll back #231. Was there a reason for that, or was it just a mistake?

Yep, my mistake... sorry... wrong version...

Corrected it too...

rollback to corrected version
@teolaz

teolaz commented Apr 18, 2018

Copy link
Copy Markdown
Author

@drewm any news on this?

@edalzell

edalzell commented Sep 6, 2019

Copy link
Copy Markdown

I would l love to see this merged...getting an errors and I can't tell why

@drzraf drzraf mentioned this pull request Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants