MLH-Fellowship / prep-project-22.OCT.PREP.1

MLH Prep Project for Pod 22.OCT.PREP.1
https://prep-project-22-oct-1.netlify.app/
MIT License
2 stars 15 forks source link

Closes #17: Adds weekly forecast #52

Open niyoj opened 1 year ago

niyoj commented 1 year ago

Changes we have done

netlify[bot] commented 1 year ago

Deploy Preview for prep-project-22-oct-1 ready!

Name Link
Latest commit c54d5b6033e578479f211d599f72764d8fc808bd
Latest deploy log https://app.netlify.com/sites/prep-project-22-oct-1/deploys/6352b5a1e1aaaf00081631bb
Deploy Preview https://deploy-preview-52--prep-project-22-oct-1.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

niyoj commented 1 year ago

@akshitadixit we have created PR that solves issue #17. I think we have the same issue of netlify deploy issues.

niyoj commented 1 year ago

@akshitadixit I have sent you the env keys in discord for the new API we used.

akshitadixit commented 1 year ago

@niyoj conflict alert 💀

niyoj commented 1 year ago

@akshitadixit solved the merge conflict

akshitadixit commented 1 year ago

@kanchitank already did that in the afternoon 🤔

kanchitank commented 1 year ago

@niyoj Unable to see the weekly forecast

image

niyoj commented 1 year ago

@kanchi let me have a look. And figure out the problem

niyoj commented 1 year ago

@akshitadixit I have solved the issue. @kanchitank can you review this PR?

niyoj commented 1 year ago

@kanchitank There was issue with that and I could not figure out why. But you can select the location on the map and the weather status will update itself.

What can we do this in this situation @akshitadixit ?

akshitadixit commented 1 year ago

Since its almost time, we could leave the most stable deployment in the repo for now.

niyoj commented 1 year ago

Okay. No problem at all @akshitadixit