{"action":"created","issue":{"url":"https://api.github.com/repos/sandstorm-io/sandstorm/issues/315","labels_url":"https://api.github.com/repos/sandstorm-io/sandstorm/issues/315/labels{/name}","comments_url":"https://api.github.com/repos/sandstorm-io/sandstorm/issues/315/comments","events_url":"https://api.github.com/repos/sandstorm-io/sandstorm/issues/315/events","html_url":"https://github.com/sandstorm-io/sandstorm/pull/315","id":65340347,"number":315,"title":"Export stats on appDemoUsers separately","user":{"login":"paulproteus","id":25457,"avatar_url":"https://avatars.githubusercontent.com/u/25457?v=3","gravatar_id":"","url":"https://api.github.com/users/paulproteus","html_url":"https://github.com/paulproteus","followers_url":"https://api.github.com/users/paulproteus/followers","following_url":"https://api.github.com/users/paulproteus/following{/other_user}","gists_url":"https://api.github.com/users/paulproteus/gists{/gist_id}","starred_url":"https://api.github.com/users/paulproteus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paulproteus/subscriptions","organizations_url":"https://api.github.com/users/paulproteus/orgs","repos_url":"https://api.github.com/users/paulproteus/repos","events_url":"https://api.github.com/users/paulproteus/events{/privacy}","received_events_url":"https://api.github.com/users/paulproteus/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-31T01:25:40Z","updated_at":"2015-04-04T00:25:40Z","closed_at":"2015-04-04T00:22:34Z","pull_request":{"url":"https://api.github.com/repos/sandstorm-io/sandstorm/pulls/315","html_url":"https://github.com/sandstorm-io/sandstorm/pull/315","diff_url":"https://github.com/sandstorm-io/sandstorm/pull/315.diff","patch_url":"https://github.com/sandstorm-io/sandstorm/pull/315.patch"},"body":"This change permits us to export appDemo user stats separately when viewing the JSON stats export.\r\n\r\n@jparyani promised me that if I pushed this, he'd make the corresponding change on the Sandstorm internal stats dashboard.\r\n\r\nFor non-internal people interested in Sandstorm stats, you might like to see how to get JSON stats data from your own Sandstorm install:\r\n\r\n* http://local.sandstorm.io:6080/stats (or whatever URL reaches your own Sandstorm)\r\n* Click on the API token at the bottom of that page\r\n* Ta-da, stats!\r\n\r\nI have tested this in the following way:\r\n\r\n* Create two demo users, each of which use an app\r\n* Create an appDemo user by visiting an appdemo URL and logging in, which also creates an instance of that app\r\n* View the mongo data on the User objects; note that there are 2 'demo' users and 1 'appDemo' user\r\n* Remove the time constraint from `cleanupExpiredUsers()` and then look at the `DeleteStats` Mongo collection and notice that there are 2 `users` and 1 `appDemoUser`s\r\n\r\nI re-did the manual test steps above, and will leave my laptop on overnight in order to trigger the real time-based `cleanupExpiredUsers()` call in the hopes of having an even more accurate test, but I already believe this is ready to merge."},"comment":{"url":"https://api.github.com/repos/sandstorm-io/sandstorm/issues/comments/89472488","html_url":"https://github.com/sandstorm-io/sandstorm/pull/315#issuecomment-89472488","issue_url":"https://api.github.com/repos/sandstorm-io/sandstorm/issues/315","id":89472488,"user":{"login":"paulproteus","id":25457,"avatar_url":"https://avatars.githubusercontent.com/u/25457?v=3","gravatar_id":"","url":"https://api.github.com/users/paulproteus","html_url":"https://github.com/paulproteus","followers_url":"https://api.github.com/users/paulproteus/followers","following_url":"https://api.github.com/users/paulproteus/following{/other_user}","gists_url":"https://api.github.com/users/paulproteus/gists{/gist_id}","starred_url":"https://api.github.com/users/paulproteus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paulproteus/subscriptions","organizations_url":"https://api.github.com/users/paulproteus/orgs","repos_url":"https://api.github.com/users/paulproteus/repos","events_url":"https://api.github.com/users/paulproteus/events{/privacy}","received_events_url":"https://api.github.com/users/paulproteus/received_events","type":"User","site_admin":false},"created_at":"2015-04-04T00:25:40Z","updated_at":"2015-04-04T00:25:40Z","body":"Yippee!\n\nOn Fri, Apr 3, 2015 at 5:22 PM, Kenton Varda <notifications@github.com>\nwrote:\n\n> Merged #315 <https://github.com/sandstorm-io/sandstorm/pull/315>.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/sandstorm-io/sandstorm/pull/315#event-273099731>.\n>\n"}}