Skip to content

Add fetcher and add 6 books#35

Open
markpotocki wants to merge 4 commits into
404minds:masterfrom
markpotocki:enders-game
Open

Add fetcher and add 6 books#35
markpotocki wants to merge 4 commits into
404minds:masterfrom
markpotocki:enders-game

Conversation

@markpotocki

Copy link
Copy Markdown
Contributor

Please check all tasks that you've completed by placing an x within the square braces below.

Proposed Changes

  • I've read the CODE OF CONDUCT and abide to it.
  • I've read the CONTRIBUTING.md
  • I've forked the repository.
  • I've created a branch and made my changes in it.
  • I understand opening a PULL REQUEST doesn't mean it will be merged for sure.

Fixes issue

Fixes #20 Fixes #18 Fixes #17 Fixes #10 Fixes #5 Fixes #4

Adds a fetcher written in go to grab book information using Google Books API. The fecther can be used by giving it a space delimited list of book titles to search for. It will grab the first result and save the information in JSON format.

Example Usage
./fetcher hunger-games alice-in-wonderland

Spaces in titles need to be separated by "-"

The json files for the remainder of the books have been generated using this. The publication dates are the dates the edition it is referencing came out not the original publication date.

FoxOfPulchritude and others added 4 commits October 1, 2019 16:18
* still need to convert to requried structure
* dates can be wrong depending on version publication
*adds Snow Crash
*adds Magic Bites
*adds Redwall
*adds Gulliver's Travels
*adds The Hunger Games
*adds Alice in Wonderland

The published date will reflect the editions publication date, not the original
@niftytushar

Copy link
Copy Markdown
Member

@markpotocki great work on the fetcher. But it gets limited information about the books, and the publisher & publication dates are also incorrect.
Is it possible to modify the fetch to also fetch data from Goodreads or Library Thing?

@markpotocki

Copy link
Copy Markdown
Contributor Author

@niftytushar it should be possible to modify it to use those I will look at their API and modify it to work with those instead. Should be able to get those in by the end of today and I’ll update this PR

@niftytushar

Copy link
Copy Markdown
Member

Awesome! Thanks you so much.

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.

Add book Magic Bites Add book Redwall Add book Gulliver's Travels Add book The Hunger Games Add book Alice in Wonderland

3 participants