Always initialize database after --config #126
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Previously, this would only run when configuring an instance for single-user usage. Now it'll also run when configuring for multi-user usage. It also adds a log when the database has already been initialized.
|
The second part of this is addressed in #127. So this PR is ready for review. |
thebaer
changed the title
|
Thanks for taking a look! Merging now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Always initialize database after --config
Previously, this would only run when configuring an instance for single-user usage. Now it'll also run when configuring for multi-user usage.
It also adds a log when the database has already been initialized.
Previous PR description:
This improves the interactive configuration process that runs with the
--configflag present -- particularly to support our work on a DigitalOcean one-click install image (T600).TODO: Support shorter config process (T657)See Shorter config process #127