-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updated forecast api #67
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
sorry for the inconvenience, i will look into it and get back to you in a bit. |
The error here is that I updated the .env file with the new key but while pushing the code, it got into .gitignore and was not pushed |
Do consider the request on the additional note above if possible. |
"aec9b7d9badbe062b1cfed513fd5903d" remove quotes aec9b7d9badbe062b1cfed513fd5903d give it directly and check it once before pushing |
can you please send me the video of working of forecast...are you running on your local before pushing..? |
The code is working just fine on my part and I have cross checked it locally twice. |
Yeah sure I will send you the video in a minute |
can we have google meet once https://meet.google.com/gcn-upit-srp join if you are free and share your screen once..i will check it once and we will fix the erros |
Pull Request
Title
Updated Forecast API which was not working properly
Description
The main error here was the placement of .env file which was not in the root directory and was causing trouble as I kept changing api's but the call was not being made.
Also the input format was not correctly followed previously as it wanted inputs as latitude and longitude not as places, for which i introduced another api to handle it.
Related Issues
None
Changes Made
Bug Fix
Checklist
Screenshots (if applicable)
Additional Notes
Please upgrade the tag for GSSoC to level-2 if possible as it took a hell lot of debugging :)