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

Weather add #13

Closed
wants to merge 9 commits into from
Closed

Weather add #13

wants to merge 9 commits into from

Conversation

antixcode6
Copy link

No description provided.

<component name="VcsDirectoryMappings">
<mapping directory="$PROJECT_DIR$" vcs="Git" />
</component>
</project>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to check in this file

@swapagarwal
Copy link
Owner

Hi @antixcode6, thank you for taking the time to add a new module.
A lot of users were querying for weather and this came up just in time! 😄
Take a look at the lines that I have commented on, and we'll be ready to merge.

@@ -1,3 +1,4 @@
ACCESS_TOKEN = '<ACCESS_TOKEN>'
VERIFY_TOKEN = '<VERIFY_TOKEN>'
WIT_AI_ACCESS_TOKEN = 'IKJJJYYVR3X672DHFVS7U7C4L2MQSS2P'
OPEN_WEATHER_MAP_ACCESS_TOKEN = 'adee94563461cdf1dcfe25956aea10ae'
Copy link
Owner

@swapagarwal swapagarwal Apr 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace the token with the placeholder, i've set it up on the server.

@swapagarwal
Copy link
Owner

Whoa, that was fast!
Two files (idea/vcs and src/movie) need to be removed and just a few more changes (see comments)

@antixcode6
Copy link
Author

Worked on the further suggestions; was just curious if module was good to go

@@ -5,7 +5,6 @@
from src import *

WIT_AI_ACCESS_TOKEN = os.environ.get('WIT_AI_ACCESS_TOKEN', config.WIT_AI_ACCESS_TOKEN)

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a blank line here to remove this file from diff

@swapagarwal
Copy link
Owner

See here https://github.com/swapagarwal/JARVIS-on-Messenger/pull/13/files
There are 6 files changed while there should be only 3.

@swapagarwal
Copy link
Owner

Why delete movie.py?

@antixcode6
Copy link
Author

Was just trying to remove it from this pull request since it had no place being there

@swapagarwal
Copy link
Owner

Can you make a new PR with changes only in the 3 required files? I think that'll be easier to merge.

@antixcode6
Copy link
Author

Alright I'm closing this I'll open a new one

@antixcode6 antixcode6 closed this Apr 28, 2016
@swapagarwal
Copy link
Owner

swapagarwal commented Apr 28, 2016

continued in #28

edadesd pushed a commit to edadesd/JARVIS-on-Messenger that referenced this pull request Jul 12, 2017
# This is the 1st commit message:

Added more tests to test_anime

# This is the commit message swapagarwal#2:

Changed Hummingbird API to Kitsu
# This is the commit message swapagarwal#3:

Added bad query test to test_anime

# This is the commit message swapagarwal#4:

Based slice length on len

# This is the commit message swapagarwal#5:

Removed repeated searches.

# This is the commit message swapagarwal#6:

Broke tests into different functions.

# This is the commit message swapagarwal#7:

Removed whitespace.

# This is the commit message swapagarwal#8:

Added tests based on API response and rating format.

# This is the commit message swapagarwal#9:

Reorganized some calls.

# This is the commit message swapagarwal#10:

Refactored API requests, added tests for rank and episode count

# This is the commit message swapagarwal#11:

Added another test case.

# This is the commit message swapagarwal#12:

Added tests for kitsu and youtube links

# This is the commit message swapagarwal#13:

Stickler corrections.

# This is the commit message swapagarwal#14:

More Stickler corrections.

# This is the commit message swapagarwal#15:

More Stickler corrections.

# This is the commit message swapagarwal#16:

Stickler corrections.

# This is the commit message swapagarwal#17:

Moved declarations closer to point used.

# This is the commit message swapagarwal#18:

Style corrections.

# This is the commit message swapagarwal#19:

Style correction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants