How to combine responses to an array using spread?

Question

I have responses from both promises but when i combineResponse it only resolve one response from ptmResponse promise , What is implemented wrong in below code using spread ? response is coming as object that i want to push it to array.

main.ts

try {
  const __data: IResponse = await makeRequest(this._request);
  const specResponse = await this.specResponse(__data.Details[0]);
  const ptmResponse = await this.ptmAccountBalanceResponse(__data.Details[1]);
  const combineResponse = {
    ...specResponse,
    ...ptmResponse
  };
  return Promise.resolve(combineResponse);
} catch (err) {
  return Promise.reject(err);
}

What are `specResponse` and `specResponse`? Objects? Arrays? — Joseph, Aug 29 '18 at 18:53
if specResponse is an empty object, for example, then it won't spread anything into combineResponse — aaron, Aug 29 '18 at 18:54
Looks like a duplicate of [this one](https://stackoverflow.com/questions/35612428/call-async-await-functions-in-parallel), no? — raina77ow, Aug 29 '18 at 18:54
@aaron i see response from both but when i combined it only resolve to one — hussain, Aug 29 '18 at 18:57
What are those objects' properties? If both have the same keys, `ptmResponse` will overwrite what's in `specResponse` — dziraf, Aug 29 '18 at 19:00

score 2 · Accepted Answer · answered Aug 29 '18 at 18:58

2

You might be overthinking this one. I'm assuming you want an array containing both objects. If so, then it's just:

const combineResponse = [ specResponse, ptmResponse ]

No spread needed.

answered Aug 29 '18 at 18:58

Joseph

117,725
30
181
234

score 0 · Answer 2 · answered Aug 29 '18 at 19:21

0

I would also fire those promises in parallel with Promise.all and store both responses in combineResponse as an array:

const combineResponse = await Promise.all([this.specResponse(__data.Details[0]), this.ptmAccountBalanceResponse(__data.Details[1])]);

answered Aug 29 '18 at 19:21

Leonid Pyrlia

1,594
2
11
14

How to combine responses to an array using spread?

2 Answers2