{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2013","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2013/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2013/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2013/events","html_url":"https://github.com/ManageIQ/manageiq/pull/2013","id":59895613,"number":2013,"title":"Extract accordion data to objects and dry up the tree and accordion logic","user":{"login":"tenderlove","id":3124,"avatar_url":"https://avatars.githubusercontent.com/u/3124?v=3","gravatar_id":"","url":"https://api.github.com/users/tenderlove","html_url":"https://github.com/tenderlove","followers_url":"https://api.github.com/users/tenderlove/followers","following_url":"https://api.github.com/users/tenderlove/following{/other_user}","gists_url":"https://api.github.com/users/tenderlove/gists{/gist_id}","starred_url":"https://api.github.com/users/tenderlove/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tenderlove/subscriptions","organizations_url":"https://api.github.com/users/tenderlove/orgs","repos_url":"https://api.github.com/users/tenderlove/repos","events_url":"https://api.github.com/users/tenderlove/events{/privacy}","received_events_url":"https://api.github.com/users/tenderlove/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/refactoring","name":"refactoring","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-03-05T02:28:07Z","updated_at":"2015-03-05T16:09:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/2013","html_url":"https://github.com/ManageIQ/manageiq/pull/2013","diff_url":"https://github.com/ManageIQ/manageiq/pull/2013.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/2013.patch"},"body":"This PR extracts accordion data to objects and removes duplicated logic.  I also added test coverage for those actions.\r\n\r\nI'm not sure if `ApplicationController::Feature` is the correct name for the class, but this is the technique we should be using."},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/77392478","html_url":"https://github.com/ManageIQ/manageiq/pull/2013#issuecomment-77392478","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2013","id":77392478,"user":{"login":"tenderlove","id":3124,"avatar_url":"https://avatars.githubusercontent.com/u/3124?v=3","gravatar_id":"","url":"https://api.github.com/users/tenderlove","html_url":"https://github.com/tenderlove","followers_url":"https://api.github.com/users/tenderlove/followers","following_url":"https://api.github.com/users/tenderlove/following{/other_user}","gists_url":"https://api.github.com/users/tenderlove/gists{/gist_id}","starred_url":"https://api.github.com/users/tenderlove/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tenderlove/subscriptions","organizations_url":"https://api.github.com/users/tenderlove/orgs","repos_url":"https://api.github.com/users/tenderlove/repos","events_url":"https://api.github.com/users/tenderlove/events{/privacy}","received_events_url":"https://api.github.com/users/tenderlove/received_events","type":"User","site_admin":false},"created_at":"2015-03-05T16:09:57Z","updated_at":"2015-03-05T16:09:57Z","body":"Ya, I am very sure we can use this same data to dry up `set_elements_and_redirect_unauthorized_user`, but that should happen in a different PR.  I'd like to get this merged, so I'm looking for feedback! :)"}}