open-sauced / hot

πŸ•The site that recommends the hottest projects on GitHub.
https://hot.opensauced.pizza
MIT License
418 stars 143 forks source link

Feature: cut over to v2 API #508

Closed jpmcb closed 7 months ago

jpmcb commented 7 months ago

Type of feature

πŸ• Feature

Current behavior

Related: https://github.com/open-sauced/api/issues/478

Suggested solution

The v1 API is being deprecated: we should cut over to the v2 API and tee up changes ready for when the v1 API goes down

Additional context

No response

Code of Conduct

Contributing Docs

takanome-dev commented 7 months ago

Would love to help with that πŸ˜ƒ Need to find my mark again πŸ˜…

jpmcb commented 7 months ago

Sure thing! Should be pretty simple for the hot repo since there's not much that's changing besides v1 to v2: here's the new API - https://alpha.api.opensauced.pizza/

We won't merge it until we've cut in the new api from the alpha branch to the beta branch but we'll want it teed up so it's ready to go.

Ping me if you have questions!

jpmcb commented 7 months ago

Hey @takanome-dev - We're getting really close to doing the v2 cutover: any progress here?

I may reassign this to me if you haven't started working on it yet since this was originally intended to be core team work and we're moving pretty fast on making this v1 --> v2 transition happen.

takanome-dev commented 7 months ago

Hey @jpmcb, I haven't started it yet. I was planning to tackle it over the weekend, as I thought there was still some time before v2 is ready. However, if it's time-sensitive for the core team, please go ahead and reassign. Apologies for any inconvenience, and thanks for the understanding!

jpmcb commented 7 months ago

No worries! I'll assign myself: we ended up moving much faster on this than I anticipated. Thanks for grabbing this tho!

jpmcb commented 7 months ago

Re-opening since we'll also want to track this going into production next week.

jpmcb commented 7 months ago

Closed by: https://github.com/open-sauced/hot/pull/510