{"action":"created","issue":{"url":"https://api.github.com/repos/thebuggenie/thebuggenie/issues/448","labels_url":"https://api.github.com/repos/thebuggenie/thebuggenie/issues/448/labels{/name}","comments_url":"https://api.github.com/repos/thebuggenie/thebuggenie/issues/448/comments","events_url":"https://api.github.com/repos/thebuggenie/thebuggenie/issues/448/events","html_url":"https://github.com/thebuggenie/thebuggenie/pull/448","id":129759518,"number":448,"title":"Work on API Overhaul","user":{"login":"founderio","id":6136865,"avatar_url":"https://avatars.githubusercontent.com/u/6136865?v=3","gravatar_id":"","url":"https://api.github.com/users/founderio","html_url":"https://github.com/founderio","followers_url":"https://api.github.com/users/founderio/followers","following_url":"https://api.github.com/users/founderio/following{/other_user}","gists_url":"https://api.github.com/users/founderio/gists{/gist_id}","starred_url":"https://api.github.com/users/founderio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/founderio/subscriptions","organizations_url":"https://api.github.com/users/founderio/orgs","repos_url":"https://api.github.com/users/founderio/repos","events_url":"https://api.github.com/users/founderio/events{/privacy}","received_events_url":"https://api.github.com/users/founderio/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2016-01-29T12:58:14Z","updated_at":"2016-01-29T23:39:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/thebuggenie/thebuggenie/pulls/448","html_url":"https://github.com/thebuggenie/thebuggenie/pull/448","diff_url":"https://github.com/thebuggenie/thebuggenie/pull/448.diff","patch_url":"https://github.com/thebuggenie/thebuggenie/pull/448.patch"},"body":"This PR includes first forays into issue [#2649](http://issues.thebuggenie.com/thebuggenie/issues/2649)\r\nIt includes output optimization of the list commands (listprojects, listissuetypes and listfieldvalues) to have better compatibility & a new route that returns a basic status overview of the bug genie.\r\nIt's only a small step but I would like to have an opinion if I am going the right way with this."},"comment":{"url":"https://api.github.com/repos/thebuggenie/thebuggenie/issues/comments/177019323","html_url":"https://github.com/thebuggenie/thebuggenie/pull/448#issuecomment-177019323","issue_url":"https://api.github.com/repos/thebuggenie/thebuggenie/issues/448","id":177019323,"user":{"login":"zegenie","id":205936,"avatar_url":"https://avatars.githubusercontent.com/u/205936?v=3","gravatar_id":"","url":"https://api.github.com/users/zegenie","html_url":"https://github.com/zegenie","followers_url":"https://api.github.com/users/zegenie/followers","following_url":"https://api.github.com/users/zegenie/following{/other_user}","gists_url":"https://api.github.com/users/zegenie/gists{/gist_id}","starred_url":"https://api.github.com/users/zegenie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zegenie/subscriptions","organizations_url":"https://api.github.com/users/zegenie/orgs","repos_url":"https://api.github.com/users/zegenie/repos","events_url":"https://api.github.com/users/zegenie/events{/privacy}","received_events_url":"https://api.github.com/users/zegenie/received_events","type":"User","site_admin":false},"created_at":"2016-01-29T23:39:01Z","updated_at":"2016-01-29T23:39:01Z","body":"First of all, thanks for taking the time to start this endeavour, it's\nsomething that is much needed. I have some specific comments on your\ncommits, but github won't let me sign in, so please take them below:\n\nI think for a first attempt, this is decent, but if I was to rewrite or\nsignificantly improve the api integration, I would go all in. Before you\nstart off, keep in mind that there is no official TBG api, so any backwards\nbreaking changes is not really a dealbreaker just yet. Hopefully, we can\nget a v1 out of this, but for now, don't worry about backwards\ncompatibility.\n\nAs for the approach with listing out details for things like issutypes,\netc, please follow the pattern from `entities\\Issue`, using a `toJSON()`\nmethod. This way, it's not up to the remote module to decide the json\nrepresentation of the objects, and it can be reused across TBG. It should\nbe possible to add a `toJSON()` method in `entities\\Keyable`, which many\nbasic datatypes inherit - adding instant json representations for most TBG\ninternal datatypes - then extend the `toJSON()` method in subclasses as\nnecessary.\n\nI very much like the idea of the status endpoint detailing latest api\nversion, but I think it can be entirely placed in the `remote` submodule,\nincluding the version numbers. To be honest, except the changes made to\nsome of the entities, there's no reason why this module cannot be external\naltogether. Let me know if you want help with that.\n\nOn Fri, Jan 29, 2016 at 1:58 PM, Oliver Kahrmann <notifications@github.com>\nwrote:\n\n> This PR includes first forays into issue #2649\n> <http://issues.thebuggenie.com/thebuggenie/issues/2649>\n> It includes output optimization of the list commands (listprojects,\n> listissuetypes and listfieldvalues) to have better compatibility & a new\n> route that returns a basic status overview of the bug genie.\n> It's only a small step but I would like to have an opinion if I am going\n> the right way with this.\n> ------------------------------\n> You can view, comment on, or merge this pull request online at:\n>\n>   https://github.com/thebuggenie/thebuggenie/pull/448\n> Commit Summary\n>\n>    - Add API version + status info (/api/:api_username/status/:format)\n>    - Clean up output of lists\n>    - Merged branch master into master\n>\n> File Changes\n>\n>    - *M* core/framework/Settings.php\n>    <https://github.com/thebuggenie/thebuggenie/pull/448/files#diff-0>\n>    (25)\n>    - *M* core/modules/remote/configuration/routes.yml\n>    <https://github.com/thebuggenie/thebuggenie/pull/448/files#diff-1> (7)\n>    - *M* core/modules/remote/controllers/Main.php\n>    <https://github.com/thebuggenie/thebuggenie/pull/448/files#diff-2>\n>    (29)\n>    - *M* core/modules/remote/templates/listissuetypes.json.php\n>    <https://github.com/thebuggenie/thebuggenie/pull/448/files#diff-3> (2)\n>    - *M* core/modules/remote/templates/listprojects.json.php\n>    <https://github.com/thebuggenie/thebuggenie/pull/448/files#diff-4> (2)\n>    - *A* core/modules/remote/templates/status.json.php\n>    <https://github.com/thebuggenie/thebuggenie/pull/448/files#diff-5> (3)\n>\n> Patch Links:\n>\n>    - https://github.com/thebuggenie/thebuggenie/pull/448.patch\n>    - https://github.com/thebuggenie/thebuggenie/pull/448.diff\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/thebuggenie/thebuggenie/pull/448>.\n>\n"}}