Closed Dieterbe closed 2 years ago
I was thinking about this one earlier. Are you okay with author pages being created with correct names (that would be my preference) or would you like the option to store them as a string w/o links?
I don't understand the question but author pages with correct names sounds great. I would not try to rearrange names, but just honor the syntax which kindle uses, which is ";" separates between names and "," is part of text and can be ignored. So it would be pages like "Vance, Ashlee", "Diamandis, Peter H" and "Kotler, Steven" ? Is there a tradeoff or downside in doing this ?
No downside, I just prefer names without the comma. I'll add an option so that I can remove the comma but have it default to the order that Kindle has it.
Thanks for the feedback!
:tada: This issue has been resolved in version 1.16.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:
I tried the new "Author First Name First" option, but it doesn't change anything for me. I even tried restarting logseq. After every re-import of highlights, the authors appear exactly in the book page medatadata as they did before. Anyway, i think this problem is orthogonal to whether first name comes first or not, as it is caused by the "," in the authors, and lack of [[ ]] around metadata values with whitespace
after checking https://docs.logseq.com/#/page/term%2Fproperties and doing some of my own experimentation, it looks like instead of formatting like so:
author:: Diamandis, Peter H.;Kotler, Steven
it should instead be:
author:: [[Diamandis, Peter H.]], [[Kotler, Steven]]
i looked at your code changes, and it seems you already worked on something along these lines. but a new import does not have any effect. it still generates the old/incorrect syntax for me. (i have plugin 1.16.7, logseq 0.8.1 )
hmm.. it seems like if i delete all the highlight pages, and then re-import everything, it works as expected. IOW. the exported pages are only truly refreshed if they are manually deleted first ?
Not sure why that's the case, I'm telling logseq to update the property on every import: https://github.com/theBenForce/logseq-plugin-my-highlights/blob/main/src/actions/syncBookHighlights.ts#L26
for example, the page
highlights/book/Abundance: The Future Is Better Than You Think
has this metadata:when I go to "all pages", in logseq, i see this: looks like all the comma characters cause a new page to be created.
The same also happens with single-author books, for example
this will create a page "Vance", and one "Ashlee"