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

(fix) Validation situations where Expo returns a 200 with an error #14

Closed

Conversation

0duaht
Copy link

@0duaht 0duaht commented Jul 4, 2018

Fixes failures with data validation cos the Hash is being converted to an array

@0duaht 0duaht changed the title (fix) Validation situations where Expo returns a 200 with an error st… (fix) Validation situations where Expo returns a 200 with an error Jul 4, 2018
Copy link

@aroeczek-old aroeczek-old left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We spotted similar errors in our app, it looks like you forgot to deal with other occurrences of response.fetch('errors').first in lines 110, 118

@0duaht
Copy link
Author

0duaht commented Aug 23, 2018

@aroeczek does 688f1bb also work with your use-case?

@aroeczek-old
Copy link

@0duaht Look like everything is ok with PR code, I didn't spot mentioned commit.

@kevinzhangftw
Copy link

yep, just test it this and it works. Can somebody from expo merge this request?

kevinzhangftw pushed a commit to nano3labs/expo-server-sdk-ruby that referenced this pull request Sep 4, 2018
@coffenbacher
Copy link

I don't understand why the first element is returned every time, it seems to me like it should always be an array rather than always a single value 🤔

#18

@lsmolic lsmolic mentioned this pull request May 28, 2019
10 tasks
@courtsimas
Copy link

any movement on this?

@pablonahuelgomez
Copy link
Contributor

Closed in favor of #29

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.

6 participants