Skip to content

Conversation

@fbartels
Copy link

@fbartels fbartels commented Jun 1, 2018

Hi,

I've found your repo through https://forum.cloudron.io/topic/1222/hackmd-fast-and-flexible-real-time-collaborative-markdown and wanted to give it a quick spin. I've standardised some paths and updated it to the latest version. As it seems it does not preserve files during upgrade I've also updated a small todo list.

fbartels and others added 30 commits June 1, 2018 16:27
Signed-off-by: Felix Bartels <felix@host-consultants.de>
add todo list

Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
set usernamefield through env variable
add comment about email env variables

Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
first step to generate user session in config.json

Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
… between app restarts)

Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
Signed-off-by: Felix Bartels <felix@host-consultants.de>
This is just a convention followed in all repos
Remove the prune because it does not work with the latest node version
* set -eu is good practice to catch scipting errors
* db is always up in cloudron case
* use exec to prevent parent shell from being around
move it to run time directory where it is not part of the backup
In addition:
* run db migration after the config files is generated and setup
* change ownership after the config is generated
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.

3 participants