Feature request: Drop `isomorphic-fetch` from implementation

contentstack / contentstack-javascript

Javascript SDK for Contentstack's Content Delivery API (Nodejs and Browser)

MIT License

26 stars 23 forks source link

Open PSoltes opened 1 week ago

PSoltes commented 1 week ago

Node 16 is no longer maintained
Node 18 onwards includes fetch by default.
Thus isomorphic-fetch should be replaced by plain fetch (either from browser or node)

Benefits

significant reduction in bundle size as isomorphic-fetch uses node-fetch which is not smallest (2.6.1 version)[https://www.npmjs.com/package/node-fetch/v/2.6.1]
isomorphic-fetch is no longer maintained

cs-raj commented 6 days ago

Hi @PSoltes thanks for raising this we will be looking into the issue and implement the required change cc: @netrajpatel

PSoltes commented 5 days ago

Thanks for considering this :+1: