news-summarizer

news-summarizer copied to clipboard

Hi, I am a hobbyist, and I am trying to change the code slightly to work for other websites, I deployed it on Vercel and added my openai api, but...

MohammedAlrozzi

changes

1

Nutlope

There is a typo in the Readme file: Use `text-curie-001` instead of `text-dacinci-003` in the `summarize` edge function should be Use `text-curie-001` instead of `text-davinci-003` in the `summarize` edge function...

gyurisc

Convert `/api/summarize` to GET endpoint so that it can use cache to reduce unnecessary OpenAI endpoint call

1

I tested locally by converting it to a GET endpoint and use query params to set URL. https://github.com/Nutlope/news-summarizer/blob/6d4814855fe9f181462777f09ec9f26a36debfbb/pages/%5B...slug%5D.tsx#L50

xcv58

UI Features and linting

1

I did more than @Nutlope explicitly asked for in https://github.com/Nutlope/news-summarizer/pull/1, but none of these are breaking changes and I think that they all add value. However, if there are objections...

skeptrunedev

1. Can it work without using self-host instead of a pricy service (e.g. Ollama + LiteLLM instead of OpenAI)? 2. Should it compare notes between news sites and concurrent events...

TomLucidor