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

2 weather #8

Closed
wants to merge 1 commit into from
Closed

2 weather #8

wants to merge 1 commit into from

Conversation

toanngosy
Copy link

I just delete one. How can I test Jarvis on my laptop? I run --- python Jarvis.py --- on Terminal and then http://localhost:5000/process/?q=<"hello"> but nothing happens. Im new to this so thanks gratefully :D

I just delete one. How can I test Jarvis on my laptop? I run --- python Jarvis.py ---  on Terminal and then http://localhost:5000/process/?q=<"hello"> but nothing happens. Im new to this so thanks gratefully :D
@swapagarwal
Copy link
Owner

swapagarwal commented Apr 22, 2016

Query like this: http://localhost:5000/process/?q=hello

@toanngosy
Copy link
Author

[toanngo@toanngo-mint] ~/JARVIS-on-Messenger (master)
❯ python jarvis.py


This is on my terminal, but there is no response. Did I do it right?

@swapagarwal
Copy link
Owner

After running python jarvis.py in the terminal, visit http://localhost:5000/process/?q=hello in a web browser. You should get a json response.

@swapagarwal swapagarwal force-pushed the master branch 2 times, most recently from 70a69c4 to 656727f Compare April 23, 2016 09:32
@swapagarwal
Copy link
Owner

Nice catch! 😄
There were some merge conflicts (I made a lot of changes here and there) so I fixed this in 656727f.
Thanks @toanngosy! 👍

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants