Skip to content

Update parse SDK to 1.9.1. #620

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

Merged

Conversation

jacob-meacham
Copy link
Contributor

The specific issue this fixes for us is parse-community/Parse-SDK-JS#311.

I've only done smoke testing of this fix, but will be deploying it to our staging instance today and will update if I run into any issues.

@facebook-github-bot
Copy link

By analyzing the blame information on this pull request, we identified @JeremyPlease, @flovilmart and @drew-gross to be potential reviewers.

@TylerBrock
Copy link
Contributor

Any updates on this one @jacob-meacham ?

@jacob-meacham
Copy link
Contributor Author

We've been using it in staging and haven't seen any issues - however, this issue was wrapped up in another issue I was having trying to wrangle binary uploads/downloads through AWS API Gateway (we've since moved away from proxying through APIG).

I believe that this PR is still required, but given the other issue I was having, it might not be.

@aontas
Copy link
Contributor

aontas commented Feb 2, 2017

This will also fix #618.

}).then(function() {
Array.prototype.slice.call(arguments).forEach(app => {
}).then(function(resolvedApps) {
Array.prototype.slice.call(resolvedApps).forEach(app => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is resolvedApps is an array, then no need to call Array.prototype.slice on it.

@jacob-meacham jacob-meacham force-pushed the feature/update-parse-sdk branch from 7991934 to 422e050 Compare March 29, 2017 05:13
@facebook-github-bot
Copy link

@jacob-meacham updated the pull request - view changes

@jacob-meacham
Copy link
Contributor Author

@flovilmart Sorry, I didn't see that this was waiting on changes from me!

@flovilmart
Copy link
Contributor

LGTM!

@flovilmart flovilmart merged commit 54cff98 into parse-community:master Apr 17, 2017
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.

5 participants