Feat/weather - Githubissues

M4-Park-Checklist / m4-parks-backend

0 stars 0 forks source link

Feat/weather #42

Closed kameronk92 closed 7 months ago

kameronk92 commented 7 months ago

Updates Made -

Type of change: added endpoint for weather forecast requests
Database/Migrations: n/a
Routes: api/v0/weather/:id
Models: forecast poro created
Controllers: weather controller added
API/JSON:
Testing:
- Models:
- Features: weather request happy and sad path
Considerations(Extra notes, refactors, etc.):

TheAveryRodriguez commented 7 months ago

This PR looks pretty solid! It's got everything we need for handling weather forecasts: a new endpoint at api/v0/weather/:id, a slick forecast PORO in the models, and a fresh weather controller to tie it all together. And hey, you even took the time to cover both the happy and sad paths in testing! Overall, this PR seems like a great addition, ticking off all the boxes for a smooth update.