{"action":"created","issue":{"url":"https://api.github.com/repos/magfest/ubersystem/issues/1133","labels_url":"https://api.github.com/repos/magfest/ubersystem/issues/1133/labels{/name}","comments_url":"https://api.github.com/repos/magfest/ubersystem/issues/1133/comments","events_url":"https://api.github.com/repos/magfest/ubersystem/issues/1133/events","html_url":"https://github.com/magfest/ubersystem/pull/1133","id":86348676,"number":1133,"title":"Insert admin user via web ui","user":{"login":"binary1230","id":5413064,"avatar_url":"https://avatars.githubusercontent.com/u/5413064?v=3","gravatar_id":"","url":"https://api.github.com/users/binary1230","html_url":"https://github.com/binary1230","followers_url":"https://api.github.com/users/binary1230/followers","following_url":"https://api.github.com/users/binary1230/following{/other_user}","gists_url":"https://api.github.com/users/binary1230/gists{/gist_id}","starred_url":"https://api.github.com/users/binary1230/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/binary1230/subscriptions","organizations_url":"https://api.github.com/users/binary1230/orgs","repos_url":"https://api.github.com/users/binary1230/repos","events_url":"https://api.github.com/users/binary1230/events{/privacy}","received_events_url":"https://api.github.com/users/binary1230/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-06-08T22:04:47Z","updated_at":"2015-06-09T14:14:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/magfest/ubersystem/pulls/1133","html_url":"https://github.com/magfest/ubersystem/pull/1133","diff_url":"https://github.com/magfest/ubersystem/pull/1133.diff","patch_url":"https://github.com/magfest/ubersystem/pull/1133.patch"},"body":"Does anyone hate this idea? I feel like I should hate myself for writing this.  Or it's brilliant.  I'm not sure.\r\n\r\nI made it so if you visit /accounts/insert_test_admin, it will do the same thing as 'sep insert_admin'\r\n\r\nIt will ONLY let you do this if there are zero admin users on the install.  Otherwise it will deny it.  So if you had an existing install and someone tried to visit this URL, it will deny them from trying to create this.\r\n\r\nEventually this could be re-worked into some kind of 'first time setup' UI or something like that.  It's a bit beyond my scope here and probably needs a lot of thinking about.\r\n\r\ndom-unstable in dockerland is pointing at this if anyone wants to check it out for themselves.  I poked it with a stick a few times it appears to behave.\r\n\r\nThere's an actual reason I wrote this too.  'docker exec sep insert_admin' doesn't have access to the DB env vars that sideboard was started with, so it can't work.  There's an easy fix there which is simply to have it connect to a host named 'db' on the psql standard port and assume a certain DB username/pass.  Separate thing."},"comment":{"url":"https://api.github.com/repos/magfest/ubersystem/issues/comments/110369087","html_url":"https://github.com/magfest/ubersystem/pull/1133#issuecomment-110369087","issue_url":"https://api.github.com/repos/magfest/ubersystem/issues/1133","id":110369087,"user":{"login":"EliAndrewC","id":651592,"avatar_url":"https://avatars.githubusercontent.com/u/651592?v=3","gravatar_id":"","url":"https://api.github.com/users/EliAndrewC","html_url":"https://github.com/EliAndrewC","followers_url":"https://api.github.com/users/EliAndrewC/followers","following_url":"https://api.github.com/users/EliAndrewC/following{/other_user}","gists_url":"https://api.github.com/users/EliAndrewC/gists{/gist_id}","starred_url":"https://api.github.com/users/EliAndrewC/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EliAndrewC/subscriptions","organizations_url":"https://api.github.com/users/EliAndrewC/orgs","repos_url":"https://api.github.com/users/EliAndrewC/repos","events_url":"https://api.github.com/users/EliAndrewC/events{/privacy}","received_events_url":"https://api.github.com/users/EliAndrewC/received_events","type":"User","site_admin":false},"created_at":"2015-06-09T14:14:29Z","updated_at":"2015-06-09T14:14:29Z","body":"Gotcha.  It sounds like there's a path to be able to use sep commands in the future, which was my main concern, so I'm +1 on this change."}}