remotion-dev / github-unwrapped-2023

Your coding year in review
https://www.githubunwrapped.com
871 stars 62 forks source link

Top language 183 #186

Closed nico-martin closed 10 months ago

nico-martin commented 10 months ago

While investigating #183 I've seen there is no way to refresh the github stats (which might already solve the issue 183). So my solution would be to allow an additional POST parameter to the api/stats endpoint called refreshCache

curl --location 'https://githubunwrapped.com/api/stats' \
--header 'Content-Type: application/json' \
--data '{"username":"nico-martin","refreshCache":true}'
JonnyBurger commented 10 months ago

Thanks a lot for implementing this! I added a limit to how often you can reset because each video rendered incurs a cost.

This is the new mechanism: githubunwrapped.com/about#private-activity

nico-martin commented 10 months ago

@JonnyBurger thanks! I see that each render costs money. Wouldn't it make sense to first show a preview of the video (running in remotion player, so no cloud render necessary) and only render once the user wants to download it? I could also imagine that a lot of people are just curious, they enter their username, then they dont like the video for what ever reason and they leave. So I guess there are a lot of renders that will never be used..