Skip to content
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

Closed
wants to merge 3 commits into from
Closed

Conversation

mishradev1
Copy link
Contributor

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

  • I have tested the changes locally
  • Documentation has been updated (if necessary)
  • Changes are backward-compatible

Screenshots (if applicable)

issue-resolved

Additional Notes

Please upgrade the tag for GSSoC to level-2 if possible as it took a hell lot of debugging :)

Copy link

vercel bot commented Oct 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agro-tech-ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2024 0:41am

Copy link
Contributor

github-actions bot commented Oct 2, 2024

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! 😊

@manikumarreddyu
Copy link
Owner

image

it is showing error like this...

@mishradev1
Copy link
Contributor Author

sorry for the inconvenience, i will look into it and get back to you in a bit.

@mishradev1
Copy link
Contributor Author

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

@mishradev1
Copy link
Contributor Author

Do consider the request on the additional note above if possible.

@manikumarreddyu
Copy link
Owner

image
it is showing same like before...remove .env from gitignore and send along with gitignore..no problem with that..

@manikumarreddyu
Copy link
Owner

"aec9b7d9badbe062b1cfed513fd5903d"

remove quotes aec9b7d9badbe062b1cfed513fd5903d give it directly and check it once before pushing

@manikumarreddyu
Copy link
Owner

can you please send me the video of working of forecast...are you running on your local before pushing..?

@mishradev1
Copy link
Contributor Author

The code is working just fine on my part and I have cross checked it locally twice.
I think the issue is arising because of the file structure of the document as .env should be in the root folder and not in src
Please look into the file structure once

@mishradev1
Copy link
Contributor Author

Yeah sure I will send you the video in a minute

@manikumarreddyu
Copy link
Owner

The code is working just fine on my part and I have cross checked it locally twice. I think the issue is arising because of the file structure of the document as .env should be in the root folder and not in src Please look into the file structure once

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

@mishradev1 mishradev1 closed this Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants