search

transitive-bullshit / nextjs-notion-starter-kit

Deploy your own Notion-powered website in minutes with Next.js and Vercel.
https://transitivebullsh.it/nextjs-notion-starter-kit
MIT License
6.28k stars 5.29k forks source link

Replace got with ky similiarly as we did in react-notion-x dependency #640

Closed onnimonni closed 2 weeks ago

onnimonni commented 2 weeks ago

Following your foot steps from https://github.com/NotionX/react-notion-x/pull/394 I think it's time to simplify the amount of http clients in the projects to just ky.

vercel[bot] commented 2 weeks ago

@onnimonni is attempting to deploy a commit to the Saasify Team on Vercel.

A member of the Team first needs to authorize it.

socket-security[bot] commented 2 weeks ago

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/ky@1.7.2 None 0 157 kB sindresorhus

🚮 Removed packages: npm/@szmarczak/http-timer@5.0.1, npm/cacheable-lookup@6.0.4, npm/form-data-encoder@1.7.1, npm/get-stream@6.0.1, npm/got@12.0.3, npm/http2-wrapper@2.1.10, npm/lowercase-keys@3.0.0, npm/p-cancelable@3.0.0

View full report↗︎

transitive-bullshit commented 2 weeks ago

Fixed in https://github.com/transitive-bullshit/nextjs-notion-starter-kit/pull/641