promise.allSettled instead of promise.all consumeAndAwaitPromises

[Ivan-Parushev]

When using the getDataFromThree method for SSR, if one query returns an error (even graphql errors like "user not authorized"), the whole SSR fails or is only partially rendered. With promise allSettled getDataFromThree will no longer break SSR because of single query. Connected issues: https://github.com/apollographql/react-apollo/issues/3678 https://github.com/apollographql/react-apollo/issues/1403

[apollo-cla]

@Ivan-Parushev: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

[Ivan-Parushev]

If somebody can assist me with Circle CI error: "FAIL ./dist/apollo-client.cjs.min.js: 24.25KB > maxSize 24.25KB (gzip)" I only changed 1 line and max size is exceeded.

[benjamn]

@Ivan-Parushev If you rebase that failure will disappear. Definitely not your fault; we were just very close to the threshold.

[cysp]

Hi, Just wondering if this change is still planned to make it into the codebase at some point? :-) I recently looked into the SSR implementation of a project I'm working on and I found the current behaviour where the entire SSR dataset is thrown away on any failure surprising, as my expectation was that a failure would only affect the request that the failure was in response to.