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

FilenamePrefix + Optional heading #5

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

felixchenier
Copy link
Contributor

Hi again @motschel123

Another PR that should be compatible with the other one (those are two distinct features so I didn't want to mix them).

  1. I added a filenameprefix option (e.g., to add a 👤 emoji in front of file name). This is interesting at least for me because I use a flat-folder form where file types are identified by emojis. I know I'm not the only one to use this kind of setup, so it may be interesting for others as well.
  2. I also changed the fullname header to be optional. In my Obsidian setup, I don't put headers in my files because I use the file name as the main header. Since there's an official option for it in Obsidian, I'm pretty sure I'm also not the only one to do so.

Feel free to merge, and if you don't that's also perfectly fine!

Regards,

Felix

…e name), and added an option to not include the name as a header.
# Conflicts:
#	src/vcard-object.ts
@motschel123
Copy link
Owner

I see where you coming from with these changes.
I will take a look at the code, but I believe that changing the file prefix in between 2 sync, will create a second set of Obsidian files.
I'll think of other ways to managing the file names.

@felixchenier
Copy link
Contributor Author

@motschel123 Yes, you're right. We'd need to set it right, once, forever. But it's the same for the folder name: if you change it, I guess a new folder will be created, duplicating all entries. A way to prevent this duplication would be to batch rename everything just after changing the settings (locate every file using the old setting, and move/rename them using the new setting). It adds complexity for a very rare use case, though. But thanks for considering!

On my side I did the change on a local repository, therefore I don't need you to do it for me, since I already have it.

Many thanks for releasing this wonderful plugin, it's really well crafted and so useful!

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

Successfully merging this pull request may close these issues.

2 participants