Skip to content

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