search

harmtemolder / koreader-calibre-plugin

A calibre plugin to synchronize metadata from KOReader to calibre
GNU General Public License v3.0
60 stars 6 forks source link

Alpha ver prep to fix reported issue with some book sync #20 #24

Closed kyxap closed 2 months ago

kyxap commented 4 months ago

I was trying to debug and find why I'm getting issue like other people https://github.com/harmtemolder/koreader-calibre-plugin/issues/20

And after zip creation from the latest everything started to work.

After digging around I've found that latest and 5.0.2-beta has some diffs:

diff -r 'C:\Users\user\AppData\Roaming\calibre\plugins\KOReader Sync' 'C:\Users\user\AppData\Roaming\calibre\plugins\KOReader Sync_'
diff -r "C:\\Users\\user\\AppData\\Roaming\\calibre\\plugins\\KOReader Sync/README.md" "C:\\Users\\user\\AppData\\Roaming\\calibre\\plugins\\KOReader Sync_/README.md"
116d115
< C
142a142,145
> ### Changed
> 
> - Removed sorting from `column_sidecar` transform because this caused an error in at least one of my books
> 
155a159,160
> - Update README to match new columns
> - Update dummy device and library to match new columns
diff -r "C:\\Users\\user\\AppData\\Roaming\\calibre\\plugins\\KOReader Sync/config.py" "C:\\Users\\user\\AppData\\Roaming\\calibre\\plugins\\KOReader Sync_/config.py"
160,161c160
<         default=str,
<         sort_keys=True
---
>         default=str

Which aligns with Unreleased

So I think it makes sense to call it alpha, give it to the people for testing and if all good we can release it as beta?

Also I've added version prefix which allow to use a/b for ui pop-up to show people this is alpha/beta version.

kyxap commented 4 months ago

Also I've uploaded zip file in forked repo if some one wants to test it: https://github.com/kyxap/koreader-calibre-plugin/releases/tag/0.5.3-alpha