jaypyles / obsidian-to-bookstack

Download and upload changes between an Obsidian Vault and a Bookstack instance.
MIT License
56 stars 6 forks source link

Script breaks when there's a / in any title #3

Open useahawk opened 9 months ago

useahawk commented 9 months ago

I have books, shelves etc which have a '/' in the title. The script breaks, since it's trying to build a path from that. I consider that a bad practice myself and have changed all the titles. Maybe titles should be escaped or theres a better way to avoid breaking the script.

Thx for your work!

Geronimoes commented 7 months ago

To expand on this issue: when Bookstack titles have other characters, such as specifically colons (':') in them, the script does work and creates a file with those characters in the file system (Ubuntu in my case). However, in my setup, I'm using Syncthing to sync my Obsidian Vault between different devices. Filenames with colons in their name don't work on Windows devices, nor on Android, it seems. Plus, I'm not sure whether Syncthing even tries to sync such files.

Not a huge issue, but many Obsidian users will have such a setup, I imagine, and especially when collaborating with other editors on a BookStack instance, it will be a challenge to enforce particular Shelf/Book/Chapter/Page titles.