omnivore-app / logseq-omnivore

Logseq plugin to fetch articles and highlights from Omnivore
MIT License
279 stars 15 forks source link

Error when syncing highlights from PDFs #45

Closed jacksonh closed 1 year ago

jacksonh commented 1 year ago

I have a feeling this is related to our parsing and sorting of patches. The patch data on a PDF is different from a web page, so this error was thrown:

Error: Invalid patch string: _<removed in case there was sensitive data
    at s.patch_fromText (index.bfcf26b1.js:2:89153)
    at c (index.bfcf26b1.js:2:91680)
    at index.bfcf26b1.js:2:166961
    at Array.sort (<anonymous>)
    at kr (index.bfcf26b1.js:2:166948)
    at async Object.loadOmnivore (index.bfcf26b1.js:2:169499)
sywhb commented 1 year ago
Screenshot 2022-08-29 at 1 26 29 PM

Fixed in v1.8.1