omnivore-app / logseq-omnivore

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

duplicate `## 🔖 Articles` node #119

Closed avelino closed 9 months ago

avelino commented 12 months ago

I don't know why, but there are some cases where the article node is duplicating

Screenshot 2023-07-07 at 12 02 00
thiswillbeyourgithub commented 12 months ago

Same

yoghurtdewdew commented 11 months ago

Same and even worse.

image image

The truth is I have deleted some ## 🔖 Articles. I just hope there will be no heading like ## 🔖 Articles there.

avelino commented 11 months ago

I noticed that it is duplicating the title article when I close line article

Screenshot 2023-07-18 at 14 46 56
yoghurtdewdew commented 11 months ago

Now it's like this

image

https://github.com/omnivore-app/logseq-omnivore/issues/119#issuecomment-1641970877

vinodpandey commented 10 months ago

For me, it was fixed by clearing local storage.

yoghurtdewdew commented 10 months ago

It doesn't work for me. I cleared local storage but it is still growing and I must clear manually.

image
vinodpandey commented 10 months ago

@yoghurtdewdew Can you please try below step and let us know the results?

image
yoghurtdewdew commented 10 months ago

Thanks for your reply! I am using the latest version, and just tried "clear cache", it doesn't work. Every time it in sync, then it will create new headings of Article.

image image
yoghurtdewdew commented 10 months ago

This is still happening 😱

image
sywhb commented 10 months ago

Sorry for the delay. I finally got time to look into this now

sywhb commented 10 months ago

Hey @avelino @yoghurtdewdew , I have found the bug that sometimes collapse::true/false will be added to the heading block after syncing or explanding/collapsing it which caused the datascript query to match the ## Articles heading to fail and create duplicate heading blocks.

I will modify the datascript to better match the heading block

sywhb commented 10 months ago

Hey @avelino @yoghurtdewdew , v1.13.4 is released to address this issue.

Please kindly delete all the duplicate heading blocks or delete this whole page in Logseq and resync with Omnivore.

Please let me know if it works.

Thank you!

yoghurtdewdew commented 10 months ago

@sywhb Thanks a lot! I found that after I changed the single page name from Omnivore to omnivore-saved, the duplication of ## Articles stopped. Hope that could be useful to the syncing fix.