omnivore-app / logseq-omnivore

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

Not embedding articles #59

Closed yosevu closed 1 year ago

yosevu commented 1 year ago

When I click the Omnivore icon to sync articles, this is the result:

Screenshot 2022-12-16 at 3 09 05 PM

I thought it might be because I use .org files, but it doesn't work when I switch to .md either.

How do I troubleshoot this issue?

Thank you!

jacksonh commented 1 year ago

Hi @yosevu, a couple questions:

  1. Do you mind taking a screenshot of your plugin settings? (Make sure not to show your API key). For example, this is what mine look like:
Screen Shot 2022-12-16 at 14 13 50

Here I mostly want to see your filter and syncAt values

  1. Do you have articles saved in Omnivore already?
yosevu commented 1 year ago

Sure! And I do have a few articles saved to Omnivore.

Also, I did see an issue about clearing the syncAt value and tried that. I will try it again just to be sure!

Settings:

Screenshot 2022-12-16 at 3 50 25 PM
jacksonh commented 1 year ago

Thanks. Usually the first thing I try when debugging any syncing issues is reseting the syncAt value. It does look like it thinks it successfully sync'd a few minutes ago (if you are in Korea timezone).

Can you set that date to something earlier than your saved articles. For example: if you just signed up yesterday, set that date toe 12/14 and then tap the Omnivore plugin button again?

yosevu commented 1 year ago

Thanks. Usually the first thing I try when debugging any syncing issues is reseting the syncAt value. It does look like it thinks it successfully sync'd a few minutes ago (if you are in Korea timezone).

Can you set that date to something earlier than your saved articles. For example: if you just signed up yesterday, set that date toe 12/14 and then tap the Omnivore plugin button again?

That did it, thanks! Syncing now. Looks like it could be a time zone related bug? I am in Korea (KST).

Additionally, it doesn't look like there is support for .org files since links and headings are parsed as markdown. I can rename the file to .md, but just curious if that is planned or something that can be contributed?

jacksonh commented 1 year ago

I suspect maybe it was related to the .org -> .md switch. I think maybe it tried to sync to .org thought it was successful so it set the syncAt value, and then when you switched, it thought it was already "caught up".

Honestly, i didn't even know about .org mode. We format everything into Markdown so will need to do a little research on this.

yosevu commented 1 year ago

Ah, I see. Well thanks for helping me figure it out! And yes Logseq supports org-mode out of the box.

jacksonh commented 1 year ago

That is really intersting, thanks. I used to use org-mode for everything in emacs.

I will close this one out and talk to @sywhb about how we should handle org-mode in the plugin. Thanks for helping debug.