https://bugs.kde.org/show_bug.cgi?id=484418

--- Comment #2 from Adam Fontenot <adam.m.fontenot+...@gmail.com> ---
Thanks for working on this! It took some effort to build against Qt6, but I did
succeed and all the books I've tried so far are working.

Couple of thoughts:

It probably makes sense to synthesize an ID field rather than using the Google
Books ones that look like "GoogleBooks:fnG8BAAAQBAJ".

The URL field has a bunch of useless stuff in it, all of the following links
for this book are previews of the book:

{http://books.google.com/books/download/Ecoregions-sample-epub.acsm?id=fnG8BAAAQBAJ&format=epub&output=acs4_fulfillment_token&dl_type=sample&source=gbs_api
http://books.google.com/books/download/Ecoregions-sample-pdf.acsm?id=fnG8BAAAQBAJ&format=pdf&output=acs4_fulfillment_token&dl_type=sample&source=gbs_api
http://play.google.com/books/reader?id=fnG8BAAAQBAJ&hl=&source=gbs_api}

It would be better to link to the official Google Books page, even if the link
has to be synthesized: https://books.google.com/books?id=fnG8BAAAQBAJ

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to