[GUI Plugin] Goodreads Sync

[CheriePie]

Further to my message above, I just discovered the error had to do with trying to set my reading progress (r_progress) column to 100 for these items. I noticed that the 100 in the error message matched that value I was trying to set for r_progress, so I removed that sync action and it went through fine.

[kiwidude]

Originally Posted by CheriePie

Further to my message above, I just discovered the error had to do with trying to set my reading progress (r_progress) column to 100 for these items. I noticed that the 100 in the error message matched that value I was trying to set for r_progress, so I removed that sync action and it went through fine.

Yes this is all why I wanted someone to test it before I released unfortunately.

Looking at the code I copied from another section and that must work in a different way. I will have to figure out a fix and push out a new version.

[kiwidude]

Release Notes:
https://github.com/kiwidude68/calibr...ds_sync-1.17.2

Fix for the setting of numeric columns (tested this time by me).

[kiwidude]

Release Notes:
https://github.com/kiwidude68/calibr...ds_sync-1.17.3

Apparently I have been flip-flopping the behaviour - supporting static values broke supporting syncing actual reading columns. Hopefully this version supports both.

[CheriePie]

Still getting the same error I posted previously with the latest 1.17.3 update when trying to update reading progress on a completed book.

calibre 9.8  embedded-python: True
Windows-11-10.0.26200-SP0 Windows ('64bit', 'WindowsPE')
('Windows', '11', '10.0.26200')
Python 3.14.2
Windows: ('11', '10.0.26200', 'SP0', 'Multiprocessor Free')
Interface language: None
EXE path: C:\Program Files\Calibre\calibre.exe
Successfully initialized third party plugins: Gather KFX-ZIP (from KFX Input) (2, 31, 0) && DeDRM (10, 0, 9) && Package KFX (from KFX Input) (2, 31, 0) && Barnes & Noble (1, 5, 8) && BookFusion Plugin (0, 8, 1) && Count Pages (1, 15, 2) && Goodreads (1, 8, 5) && Goodreads Sync (1, 17, 3) && KFX metadata reader (from KFX Input) (2, 31, 0) && From KFX (2, 31, 0) && KFX Input (2, 31, 0) && Set KFX metadata (from KFX Output) (2, 19, 0) && KFX Output (2, 19, 0) && Kindle hi-res covers (0, 5, 1) && Quality Check (1, 14, 7)
Traceback (most recent call last):
  File "calibre_plugins.goodreads_sync.dialogs", line 810, in action_button_clicked
  File "calibre_plugins.goodreads_sync.dialogs", line 834, in _apply_add_actions_for_books
  File "calibre_plugins.goodreads_sync.core", line 1083, in apply_actions_to_calibre
  File "calibre_plugins.goodreads_sync.core", line 1220, in _apply_custom_column_changes_to_books
KeyError: '100'

so I went ahead and removed that rule I created to update the r_progress field again to push it through and it was successful.

[kiwidude]

Release Notes:
https://github.com/kiwidude68/calibr...ds_sync-1.17.4

This is the final attempt at this - this time around I have tested all the permutations I could see of updating reading progress vs syncing with a manual value applied. I don't use the sync or reading progress features personally normally, but enough "quick fixes" already...

[CheriePie]

No worries... if it doesn't work, just revert it back to how it was before. I certainly didn't expect to make all this extra work for you, and while I appreciate it, I don't want to see you so exasperated with all the wickedness of it! LOL

I'll try it out tomorrow. On my way to bed now.