search

MLH-Fellowship / prep-project-4.1.1

MLH Prep Project for Pod 4.1.1
https://mlh-prep-4-1-1.netlify.app/
MIT License
6 stars 13 forks source link

WIP : Hourly Forecasts #20

Open chiraag-kakar opened 2 years ago

chiraag-kakar commented 2 years ago

Draft pull request by @chiraag-kakar and @gavishpoddar for #1 .

netlify[bot] commented 2 years ago

✔️ Deploy Preview for mlh-prep-4-1-1 ready!

🔨 Explore the source changes: f8b771d44ed86cda9d081be4c371bcbfdd856af2

🔍 Inspect the deploy log: https://app.netlify.com/sites/mlh-prep-4-1-1/deploys/616d65fef7474f000720c23c

😎 Browse the preview: https://deploy-preview-20--mlh-prep-4-1-1.netlify.app/

khattakdev commented 2 years ago

@chiraag-kakar @gavishpoddar I don't see any changes in this PR.

What's the progress?

gavishpoddar commented 2 years ago

Hi @khattakdev, can you please review this PR (and possibly merge it)

khattakdev commented 2 years ago

@gavishpoddar The Call is returning 401. image

gavishpoddar commented 2 years ago

Hey, @khattakdev I have fixed that issue. Another issue is showing.

Screenshot 2021-10-18 at 5 53 09 PM
khattakdev commented 2 years ago
Screenshot 2021-10-18 at 5 53 09 PM

This issue isn't from your side. The deploy preview uses Will's API which has exceeded the API limit

Swatilekha-Roy commented 2 years ago

@chiraag-kakar @gavishpoddar could you please resolve the conflicts in this pr, so that I could review it for you? Thanks.

gavishpoddar commented 2 years ago

Hi, @Harshal0902 @Swatilekha-Roy, Thank you so much for the review 😄. But an equivalent PR has already been merged #30.