search

duydl / logseq-calibre-annotation

Other
20 stars 2 forks source link

Issues with local copy of master #10

Open Victor239 opened 2 months ago

Victor239 commented 2 months ago

I'm trying to add a feature to your plugin but I'm having issues with it, I'm not sure what I might be doing wrong as I'm new to Logseq plugin development.

In the console I get these errors when I load the unpacked plugin (using the master branch):

Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('file://') does not match the recipient window's origin ('null').

lsplugin.core.js:2 ERROR: [Loader:#logseq-calibre-metadata - logseq-calibre-metadata][16:27:26] loadhandshake Timeout Error: handshake Timeout
    at file:///app/logseq/resources/app/js/lsplugin.core.js:2:88419
duydl commented 2 months ago

From my experience handshake timeout could happen but the loading would repeat again. Most of the time we could still load the plugin eventually, maybe a reload is required.

Btw, I am planning to migrate calibre-metadata to calibre-annotation to unify them into one plugin this month. This may affect your work.

duydl commented 2 months ago

This is the branch of migration https://github.com/duydl/logseq-calibre-annotation/tree/merge-calibre-metadata . If you still have the problem, you may consider make a PR so it would be possible for me to help.