-
Notifications
You must be signed in to change notification settings - Fork 981
Weather module #76
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
Weather module #76
Conversation
|
@williamshoops96 Welcome to the world of open-source. Your PR looks quite good for a first contribution! 😄 |
|
Remove all the .DS_Store files. There should be changes in 4 files only: https://github.com/swapagarwal/JARVIS-on-Messenger/pull/76/files |
| 'request', | ||
| 'time', | ||
| 'wiki', | ||
| 'weather', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alphabetical
|
Thank you haha! Program a lot, just trying to become better by
|
|
And yeah sorry about the Ds store. Developing on mac, need to remember to
not commit those.
|
| 'time', | ||
| 'weather', | ||
| 'wiki', | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extra line
|
Maybe this can be of help: https://jonbellah.com/recursively-remove-ds-store/ |
|
Ah thanks. Will make it a lot faster!
|
|
reference #79 |
So I've adapted the original p #28 r and got an account on openweathermap to use the api key. I think, the intents need to be added on wit.ai but I'm not sure. Info on this? It's one of my first proper contributions to a project!