Twaha-Rahman/e-commercify

[REQUEST] Agree on a fetch package

Closed this issue · 3 comments

Twaha-Rahman commented

Is your feature request related to a problem? Please describe.

We'll start writing tests for our completed endpoints and for that we'll need to send a request somehow. I know that Node.js has HTTPS as a standard library that can be used for this sort of stuff but it's quite low-level and doesn't support HTTPS by default. That's why I'm asking that we agree on a popular fetch package from NPM.

Describe the solution you would like

I'd like to suggest node-fetch as a really competent library. If we use this library, we'll be able to use the almost identical API in the frontend too.

Describe alternatives you have considered

Use theHTTPS standard library provided with Node.js

issue-label-bot commented

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.80. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

  • 👍1
snowteamer commented

I have used the ´request` library in the past but it is no longer recommended, so I will agree on node-fetch.

  • 👍1
github-actions commented

This issue has been inactive for a while. Adding the Stale label. 😕