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

Write note to filesystem if file exists #1504

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

daniele-athome
Copy link
Contributor

This patch writes note content back to the filesystem if a file exists in the public docs path. This can be a first step towards the real objective of #90 maybe?

Please note that this doesn't address the fact that notes loaded from docsPath are set with a null user owner ID so they are not actually editable. You need to update the row in the database for that. I believe that part should be addressed in another way (I was thinking something like per-user docsPath)

Signed-off-by: Daniele Ricci <daniele@casaricci.it>
@jackycute
Copy link
Member

Hi @daniele-athome,
I think this is great! It will be better if there is a config to enable/disable this feature.

@daniele-athome
Copy link
Contributor Author

I'm sorry but I stopped using CodiMD 😢

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.

None yet

2 participants