Open jaredly opened 11 years ago
Yes, this comes back to something I've wanted to do for a while which is a proper onboarding flow - setting up the first user, database url, and adding the first repo.
Hmmm database url? So setting an ENV variable? or writing to a config file?
On Fri, Jul 26, 2013 at 11:32 AM, Peter Braden notifications@github.comwrote:
Yes, this comes back to something I've wanted to do for a while which is a proper onboarding flow - setting up the first user, database url, and adding the first repo.
— Reply to this email directly or view it on GitHubhttps://github.com/Strider-CD/strider/issues/149#issuecomment-21635919 .
I envisage something like the web-based wizard you get on a vanilla WordPress install.
We'd have a flow for:
I actually had an app which would do this, but it was for an older version of Strider (back when we had a config file). All you had to do was enter your Heroku API key and it could fully install & configure a Strider instance for you:
Right; wordpress uses a file to store config.
And I think that would be totally appropriate for us too; there will be a number of configuration options, and I'd rather have them in a file than having tons of ENV vbls.
I think STRIDER_CONFIG_FILE could be in env, defaulting to something reasonable.
On Fri, Jul 26, 2013 at 1:10 PM, niallo notifications@github.com wrote:
I envisage something like the web-based wizard you get on a vanilla WordPress install.
We'd have a flow for:
- Adding admin user
- Setting up database (perhaps even provisioning, on a platform like Heroku)
- Configuring OAuth for Github
- Etc
I actually had an app which would do this, but it was for an older version of Strider (back when we had a config file). All you had to do was enter your Heroku API key and it could fully install & configure a Strider instance for you:
https://github.com/niallo/deadlift
— Reply to this email directly or view it on GitHubhttps://github.com/Strider-CD/strider/issues/149#issuecomment-21641873 .
Yeah, we should have an environment variable and a -c [config file]
flag on the CLI interface would be nice too.
Sounds great. So what format do we like best? javascript? or json/yaml?
On Fri, Jul 26, 2013 at 5:08 PM, niallo notifications@github.com wrote:
Yeah, we should have an environment variable and a -c [config file] flag on the CLI interface would be nice too.
— Reply to this email directly or view it on GitHubhttps://github.com/Strider-CD/strider/issues/149#issuecomment-21653573 .
Problem with json is it doesn't allow comments unfortunately. Not the biggest fan of yaml but it might be the right choice here.
Sent from my iPhone
On Jul 26, 2013, at 8:35 PM, Jared Forsyth notifications@github.com wrote:
Sounds great. So what format do we like best? javascript? or json/yaml?
On Fri, Jul 26, 2013 at 5:08 PM, niallo notifications@github.com wrote:
Yeah, we should have an environment variable and a -c [config file] flag on the CLI interface would be nice too.
— Reply to this email directly or view it on GitHubhttps://github.com/Strider-CD/strider/issues/149#issuecomment-21653573 .
— Reply to this email directly or view it on GitHub.
Json for sure
Sent from my iPhone
On Jul 26, 2013, at 8:35 PM, Jared Forsyth notifications@github.com wrote:
Sounds great. So what format do we like best? javascript? or json/yaml?
On Fri, Jul 26, 2013 at 5:08 PM, niallo notifications@github.com wrote:
Yeah, we should have an environment variable and a -c [config file] flag on the CLI interface would be nice too.
— Reply to this email directly or view it on GitHubhttps://github.com/Strider-CD/strider/issues/149#issuecomment-21653573 .
— Reply to this email directly or view it on GitHub.
What about using something like json-minify? https://github.com/getify/JSON.minify That would give us comments.
On Fri, Jul 26, 2013 at 9:39 PM, Peter Braden notifications@github.comwrote:
Json for sure
Sent from my iPhone
On Jul 26, 2013, at 8:35 PM, Jared Forsyth notifications@github.com wrote:
Sounds great. So what format do we like best? javascript? or json/yaml?
On Fri, Jul 26, 2013 at 5:08 PM, niallo notifications@github.com wrote:
Yeah, we should have an environment variable and a -c [config file] flag on the CLI interface would be nice too.
— Reply to this email directly or view it on GitHub< https://github.com/Strider-CD/strider/issues/149#issuecomment-21653573> .
— Reply to this email directly or view it on GitHub.
— Reply to this email directly or view it on GitHubhttps://github.com/Strider-CD/strider/issues/149#issuecomment-21658776 .
+1
Sent from my iPhone
On Jul 26, 2013, at 8:49 PM, Jared Forsyth notifications@github.com wrote:
What about using something like json-minify? https://github.com/getify/JSON.minify That would give us comments.
On Fri, Jul 26, 2013 at 9:39 PM, Peter Braden notifications@github.comwrote:
Json for sure
Sent from my iPhone
On Jul 26, 2013, at 8:35 PM, Jared Forsyth notifications@github.com wrote:
Sounds great. So what format do we like best? javascript? or json/yaml?
On Fri, Jul 26, 2013 at 5:08 PM, niallo notifications@github.com wrote:
Yeah, we should have an environment variable and a -c [config file] flag on the CLI interface would be nice too.
— Reply to this email directly or view it on GitHub< https://github.com/Strider-CD/strider/issues/149#issuecomment-21653573> .
— Reply to this email directly or view it on GitHub.
— Reply to this email directly or view it on GitHubhttps://github.com/Strider-CD/strider/issues/149#issuecomment-21658776 .
— Reply to this email directly or view it on GitHub.
+2
node-config with json5
To make things even easier, perhaps we could show a "create admin user" screen -- if there are zero users in the database (which would only be the very first time it's run).