![]() |
#16 | |
Connoisseur
![]() Posts: 55
Karma: 10
Join Date: Apr 2010
Location: new york city
Device: nook, ipad
|
Quote:
just sent you a PM. Thanks for looking into this! |
|
![]() |
![]() |
![]() |
#17 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,667
Karma: 2162246
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
The problem sdow1 was having is another recurrence of a bug in Goodreads that I have had to "enhance" my workaround for. It is caused by a small minority of books having invalid data being sent through the Goodreads API. The next version of the plugin will now ignore these books so you can sync the rest of your shelf, and hopefully Goodreads will address the issue soon anyways.
The next version also has the implementation of retrieving tags from your shelves using a customisable mapping as requested by msr. I've restructured the menu a little to accommodate this. I can envisage other actions in future appearing on the linked book menu like creating reviews. See the screenshots for how this appears. The other addition to the next version is changing where the goodreads linked id is stored to the new identifiers table Kovid has added to the next release. This is working really well and it is great to be able to have a custom column either displaying the id or a Y/N type flag if you want it to show which books you have linked. If anyone is willing to test the next version for me you will need to be running from source code to get Kovid's changes until the next Calibre release. Drop me a PM if you want to give it a go, or else wait a few days for an official release. |
![]() |
![]() |
Advert | |
|
![]() |
#18 |
Member
![]() Posts: 18
Karma: 10
Join Date: Feb 2011
Device: Nook
|
Went to link a book this morning and got:
calibre, version 0.7.48 ERROR: Unhandled exception: <b>TypeError</b>:arguments did not match any overloaded call: QLineEdit(QWidget parent=None): too many arguments QLineEdit(QString, QWidget parent=None): argument 1 has unexpected type 'NoneType' Traceback (most recent call last): File "C:\Users\dwarren\AppData\Roaming\calibre\plugins\ Goodreads Sync.zip\goodreads_sync_code.py", line 2627, in search_goodreads_to_link_book File "C:\Users\dwarren\AppData\Roaming\calibre\plugins\ Goodreads Sync.zip\goodreads_sync_code.py", line 2665, in search_to_link_books File "C:\Users\dwarren\AppData\Roaming\calibre\plugins\ Goodreads Sync.zip\goodreads_sync_code.py", line 492, in __init__ File "C:\Users\dwarren\AppData\Roaming\calibre\plugins\ Goodreads Sync.zip\goodreads_sync_code.py", line 174, in __init__ TypeError: arguments did not match any overloaded call: QLineEdit(QWidget parent=None): too many arguments QLineEdit(QString, QWidget parent=None): argument 1 has unexpected type 'NoneType' This is based off the version you E-mailed me last week, and 0.7.48 |
![]() |
![]() |
![]() |
#19 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,667
Karma: 2162246
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
You are using an "old" preview of the next release. Now you are on 0.7.48 I will email you the latest version of it.
I don't want to make it publicly available as yet because there is one last thing I want to do with it. (Goodreads have said that as of tomorrow a new API method will be available which will allow when adding new shelves to specify more options than just a name.) |
![]() |
![]() |
![]() |
#20 |
Connoisseur
![]() Posts: 55
Karma: 10
Join Date: Apr 2010
Location: new york city
Device: nook, ipad
|
Hey kiwidude:
I just synced from goodreads this morning since I had added a few new books to my wishlist, and the problem book that we had decided to skip suddenly started sync-ing. I can only assume this is because goodreads fixed something on their end, given that I haven't updated the plugin since our last discussion. Perhaps it has something to do with the updated API they were promising? |
![]() |
![]() |
Advert | |
|
![]() |
#21 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,667
Karma: 2162246
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
Yes that is exactly right, I was going to drop you a note about that. Goodreads did a website software update yesterday and I tested it very late last night to find certainly for that particular book the issue is resolved (and hopefully for any others that had the same problem as well).
I am very close to releasing the next version of this plugin, I just have a couple of people testing it at the moment and waiting to hear if they have any issues with it. |
![]() |
![]() |
![]() |
#22 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,667
Karma: 2162246
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
v1.1 Released
Download from the first post on this thread - note that this requires at least the latest version of Calibre of v0.7.48. To the few of you who I sent test versions, please ensure you grab this official release version.
New features:
|
![]() |
![]() |
![]() |
#23 |
Member
![]() Posts: 23
Karma: 10
Join Date: Jul 2009
Device: Kindle2
|
I really like the display column for whether a book is linked or not.
Getting an out-of-index error (see below) when I try to download tags. Not sure where the error is likely to be (other than due to my plethora of shelves). Tried a couple of different linked books and get the same error each time. Code:
calibre, version 0.7.48 ERROR: Unhandled exception: <b>IndexError</b>:list index out of range Traceback (most recent call last): File "C:\Users\Mike\AppData\Roaming\calibre\plugins\Goodreads Sync.zip\goodreads_sync_code.py", line 3319, in download_tags File "site-packages\calibre\library\database2.py", line 200, in get_property File "site-packages\calibre\library\caches.py", line 211, in __getitem__ IndexError: list index out of range |
![]() |
![]() |
![]() |
#24 |
I <3 my Kindle
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 528
Karma: 51332
Join Date: Nov 2010
Location: United States
Device: Kindle 3G + WiFi
|
Cool, thanks! I'm new to this extension, and it took me a while to figure out that I had to add the toolbar button. Once I got that, the whole thing was a breeze. I especially like how you use the API instead of just storing the username/password (vulnerable) and "scraping" off the website.
Since I have ~50 books in my Calibre library, it took a while to link every book. It's not a big deal though, since I only have to do it once. However, if you want to fix the problem, I suggest that you instead have one main window that lists all the books that I selected. Books that yield one result are auto-selected, so that people will only have to fine-tune their books that have many different search results. (If that makes any sense ![]() |
![]() |
![]() |
![]() |
#25 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,667
Karma: 2162246
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
v1.1.1 Released
Sorry folks, something was broken in yesterdays release as reported by msr above for downloading tags. New version attached to the first post.
|
![]() |
![]() |
![]() |
#26 | |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,667
Karma: 2162246
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
Quote:
|
|
![]() |
![]() |
![]() |
#27 | |
I <3 my Kindle
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 528
Karma: 51332
Join Date: Nov 2010
Location: United States
Device: Kindle 3G + WiFi
|
Quote:
|
|
![]() |
![]() |
![]() |
#28 |
Enthusiast
![]() ![]() Posts: 27
Karma: 100
Join Date: May 2010
Device: none
|
Jut wanted to add a quick thank-you.
I'm some way from being organised enough to use this plug-in, but your other two are wicked useful. When I first started reading the description, I was afraid it wouldn't be able to handle a high volume - glad to see there is a CSV option. Looking forward to getting stuck into this. |
![]() |
![]() |
![]() |
#29 | ||
US Navy, Retired
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,865
Karma: 13806776
Join Date: Feb 2009
Location: North Carolina
Device: Icarus Illumina XL HD, Nexus 7
|
Quote:
Quote:
|
||
![]() |
![]() |
![]() |
#30 |
Enthusiast
![]() ![]() Posts: 27
Karma: 100
Join Date: May 2010
Device: none
|
@dwanthny
Thank you for the clarification... ![]() Not to worry, that is what I was referring to... |
![]() |
![]() |
![]() |
Tags |
calibre, goodreads |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
[GUI Plugin] Clipboard Search | kiwidude | Plugins | 29 | 04-02-2024 10:05 PM |
[GUI Plugin] Search the Internet | kiwidude | Plugins | 433 | 04-01-2024 05:48 PM |
[GUI Plugin] Open With | kiwidude | Plugins | 403 | 04-01-2024 08:39 AM |
[GUI Plugin] Quick Preferences | kiwidude | Plugins | 62 | 03-16-2024 11:47 PM |
[GUI Plugin] Book Sync **Deprecated** | kiwidude | Plugins | 111 | 06-07-2011 07:47 PM |