Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when importing tags with spaces in a namespace #210

Open
logseq-cldwalker opened this issue Jan 23, 2025 · 2 comments
Open

Error when importing tags with spaces in a namespace #210

logseq-cldwalker opened this issue Jan 23, 2025 · 2 comments
Assignees

Comments

@logseq-cldwalker
Copy link
Contributor

I can confirm that this fix greatly reduced the number of "No uuid" errors for me on import. I still get 15 but 13 of those are from the metadata created during Zotero imports. Specifically, I have page properties in the MD document that look like this:

tags:: [[Social Science / Anthropology / Cultural & Social]]

tags:: [[History / Asia / Japan]], [[Social Science / Indigenous Studies]], [[✅]]

tags:: [[/unread]], [[Philosophy / Political]], [[Political Science / History & Theory]], [[Social Science / Sociology / Social Theory]]

tags:: [[/unread]], [[Art / Australian & Oceanian]], [[Social Science / Anthropology / Cultural & Social]], [[Social Science / Media Studies]]

To just give a few examples. These tags make sense in Zotero, but cause problems in Logseq when trying to import.

Originally posted by @kerim in #191

@logseq-cldwalker logseq-cldwalker changed the title Error when importing tags with namespaces Error when importing tags with spaces in a namespace Jan 23, 2025
@logseq-cldwalker logseq-cldwalker self-assigned this Jan 23, 2025
logseq-cldwalker added a commit to logseq/logseq that referenced this issue Jan 23, 2025
Fixes logseq/db-test#210. Also fix invalid
EDN class db-ident related to this
@logseq-cldwalker
Copy link
Contributor Author

@kerim Thanks for the bug report. I split this out into a separate sub-issue since this is caused by a different bug. I've pushed a fix up to test.logseq.com which should be ready in ~10m. Let me know how it goes

@kerim
Copy link

kerim commented Jan 24, 2025

Thanks. I'm traveling right now so may not be able to test till next week...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants