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

Add book module #11

Merged
merged 4 commits into from Apr 23, 2016
Merged

Add book module #11

merged 4 commits into from Apr 23, 2016

Conversation

havanagrawal
Copy link
Contributor

This module is identical in structure and working to all the other ones.. Basically makes a call to the Goodreads API (You can sign up for a Goodreads Developer Key here

Small difference is that the api does not offer json output, so I've used some minor xml parsing to get the data.

Updates intents and entities on https://wit.ai/havanagrawal/JARVIS-on-Messenger

Sample output
book_usage

@@ -1,3 +1,4 @@
ACCESS_TOKEN = '<ACCESS_TOKEN>'
VERIFY_TOKEN = '<VERIFY_TOKEN>'
WIT_AI_ACCESS_TOKEN = 'IKJJJYYVR3X672DHFVS7U7C4L2MQSS2P'
GOODREADS_ACCESS_TOKEN = '<GOODREAD_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.

*GOODREADS

@swapagarwal
Copy link
Owner

That is one awesome module you built there!
I've set up both goodreads and wit on my side. Can you add some tests so that we can see it in action?

@havanagrawal
Copy link
Contributor Author

Anything else I need to do here?

@swapagarwal
Copy link
Owner

Nope, looks perfect! (Sorry, I got caught up fixing a bug so I didn't see this earlier.)

@swapagarwal swapagarwal merged commit 91d7039 into swapagarwal:master Apr 23, 2016
@swapagarwal
Copy link
Owner

Deployed, and working as expected. Try it out with "anything you want book".
Thanks @havanagrawal! 👍

@havanagrawal havanagrawal deleted the book_feature branch May 1, 2016 05:30
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