readwiseio / logseq-readwise-official-plugin

GNU General Public License v3.0
56 stars 3 forks source link

Tweets with bullet points are cut short #2

Closed pamil closed 1 year ago

pamil commented 2 years ago

Eg. this one: https://twitter.com/Mappletons/status/1265608212528717825

On Readwise:

CleanShot 2022-07-06 at 00 48 08@2x

Synced with LogSeq:

CleanShot 2022-07-06 at 00 48 52@2x
brunojm commented 2 years ago

Thanks for reporting @pamil, I'm on it.

brunojm commented 1 year ago

Hey @pamil this seems a bug on Logseq itself, I'm going to create an issue on their repo.

pstuifzand commented 1 year ago

I found out that for the Logseq parser blocks can't start with a dash and also lines can't start with a dash. A solution would be to escape - with \\. Another solution could be to replace - with *, but that changes the message.

https://github.com/logseq/logseq/issues/5664

brunojm commented 1 year ago

Fixed on https://github.com/readwiseio/logseq-readwise-official-plugin/commit/ad313b4255dad6c3478fb0938473111f00f4d52f

pstuifzand commented 1 year ago

@brunojm This fix creates a problem with - dashes on the line itself. Example is a property full-title:: Title, which is imported as: full\-title:: Title. Logseq shows an error for this property.