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 and Example Script #20

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Conversation

andykemp
Copy link

@andykemp andykemp commented Jan 5, 2018

I've added an example script which uses the framework to create a separate .txt file for each book in markdown format which includes all quotes, page numbers, notes and generates a kindle:// link to the highlight.

I use this to save to my dropbox folder and then display in Uylsses as a writing prompt.

Extract the page number (where available) and the contents of any notes recorded with that highlight.

highlight.page will return "Page: 7" or "Location: 123" depending on whether the page number is available...
Updated test data to include highlight.page and highlight.note
Update fetching_books_and_highlights_test.rb
Fixed test files to include notes and page data
Updated to include details of highlight.page and highlight.note
Added description of script to start
@andykemp
Copy link
Author

andykemp commented Jan 5, 2018

Should say this is reliant on changes in my other PR.

@andykemp andykemp closed this Jan 6, 2018
@andykemp andykemp reopened this Jan 6, 2018
@speric
Copy link
Owner

speric commented Jan 9, 2018

I left some feedback on #19, some of which I now realize I should have left here.

Either way, my general comments remain the same.

Thanks for these PRs. 👊

@andykemp
Copy link
Author

Resolved in other PR

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