Lesson 14: const { responses: imageUploadResponses, files } - doubt

[ManuWeb3]

Hi all,

What is the purpose of responses: imageUploadResponses in the below code line in the script: 03-deploy-random-ipfs-nft.js

const { responses: imageUploadResponses, files } = await storeImages(imagesLocation)

Are we assigning responses to imageUploadResponses by using some kind of JS operator : ?

If yes, then why could we not use responses and, let's say, responsesIndex variables directly instead of imageUploadResponses and imageUploadResponsesIndex in the for() loop ?

[RoboCrypter]

@ManuWeb3 : The purpose of responses: imageUploadResponses, files is :

In our storeImages folder which is in our uploadToPinata folder,

So, In that function return statement there are two things responses, files,

In responses which is an array has imageUploadResponses thats why we wrote like responses: imageUploadResponses because we want those Upload Responses from inside responses and , files

[alymurtazamemon]

@abossofmyself Good!

[alymurtazamemon]

@ManuWeb3 from the storeImages function we are returning an object with variables names { responses, files }, just like we do in the hardhat helper file. And in the handleTokenUris function we are destructuring (JavaScript feature) it const { responses: imageUploadResponses, files } and also renamed the first parameter. Just like we do in the imports

const {
  storeImages,
  storeTokeUriMetadata,
} = require("../utils/uploadToPinata");

[krakxn]

It is actually really simple:

const { responses: imageUploadResponses, files } = await storeImages(imagesLocation)

From storeImages(imagesLocation) we are extracting responses.

Then, we are renaming that responses to imageUploadResponses via responses: imageUploadResponses.

Why? For convenience and aptness in the specific context which makes it much easier to comprehend.

[ManuWeb3]

@krakxn thanks. Got it now.
I had a hunch that we're kind of assigning (renaming, you put it better) responses to imageUploadResponses but then I wasn't sure why we need that, and as you said, to make it more context-apt.

Also just ran the script by removing imageUploadResponses and imageUploadResponsesIndex and rather used responses and responsesIndex and it worked fine!!

[krakxn]

Awesome! The more you know via self-learning :)