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

Add reminder that data/ directory needs to be writable to README #11

Merged
merged 1 commit into from
Feb 15, 2019

Conversation

freefal
Copy link
Contributor

@freefal freefal commented Feb 14, 2019

I installed todomini (which is great, btw) on my server and found that my edits were not surviving a browser refresh. Making the data/ directory writable by all users fixed the problem, by allowing my edits to be persisted on the server.

Of course, the more correct way to do this is to ensure that the user under which PHP is running has write permission to the data/ directory and I've added this suggestion to the README.

I installed todomini (which is great, btw) on my server and found that my edits were not surviving a browser refresh. Making the `data/` directory writable by all users fixed the problem, but allowing my edits to be persisted on the server.

Of course, the more correct way to do this is to ensure that the user under which PHP is running has write permission to the `data/` directory and I've added this suggestion to the README.
@freefal freefal changed the title Add warning that data/ directory needs to be writable Add reminder that data/ directory needs to be writable to README Feb 14, 2019
@chr15m chr15m merged commit a8f07d0 into AppMini:master Feb 15, 2019
@chr15m
Copy link
Member

chr15m commented Feb 15, 2019

Thank you very much!

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