{"action":"created","issue":{"url":"https://api.github.com/repos/ejeschke/ginga/issues/407","repository_url":"https://api.github.com/repos/ejeschke/ginga","labels_url":"https://api.github.com/repos/ejeschke/ginga/issues/407/labels{/name}","comments_url":"https://api.github.com/repos/ejeschke/ginga/issues/407/comments","events_url":"https://api.github.com/repos/ejeschke/ginga/issues/407/events","html_url":"https://github.com/ejeschke/ginga/pull/407","id":191171456,"number":407,"title":"Refactor reference viewer","user":{"login":"ejeschke","id":1305473,"avatar_url":"https://avatars.githubusercontent.com/u/1305473?v=3","gravatar_id":"","url":"https://api.github.com/users/ejeschke","html_url":"https://github.com/ejeschke","followers_url":"https://api.github.com/users/ejeschke/followers","following_url":"https://api.github.com/users/ejeschke/following{/other_user}","gists_url":"https://api.github.com/users/ejeschke/gists{/gist_id}","starred_url":"https://api.github.com/users/ejeschke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejeschke/subscriptions","organizations_url":"https://api.github.com/users/ejeschke/orgs","repos_url":"https://api.github.com/users/ejeschke/repos","events_url":"https://api.github.com/users/ejeschke/events{/privacy}","received_events_url":"https://api.github.com/users/ejeschke/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2016-11-23T01:52:32Z","updated_at":"2016-11-29T20:42:03Z","closed_at":"2016-11-29T20:32:14Z","pull_request":{"url":"https://api.github.com/repos/ejeschke/ginga/pulls/407","html_url":"https://github.com/ejeschke/ginga/pull/407","diff_url":"https://github.com/ejeschke/ginga/pull/407.diff","patch_url":"https://github.com/ejeschke/ginga/pull/407.patch"},"body":"This plugin reorganizes and consolidates the reference viewer code under `ginga.rv`.\r\n\r\nThis will aid in documenting and understanding how the reference viewer is put together and separates it a bit from the rest of the ginga toolkit.  As part of this, I have also merged two classes that used to be combined at run-time to produce the reference viewer shell.  For a long time now this has not been necessary and so it makes sense to wrap that all up together in one class for clarity.  I've also moved the `Channel` class into it's own file for the same reason.\r\n\r\nThe plugins are reorganized into one subfolder `ginga.rv.plugins`. This mimics the design of custom reference viewers that do not subclass ginga's reference viewer.\r\n\r\nThis reorganization should not affect any users except those creating a custom reference viewer by subclassing ginga's, or are subclassing an existing plugin.  For these users a simple path change importing the module is all that is needed."},"comment":{"url":"https://api.github.com/repos/ejeschke/ginga/issues/comments/263692435","html_url":"https://github.com/ejeschke/ginga/pull/407#issuecomment-263692435","issue_url":"https://api.github.com/repos/ejeschke/ginga/issues/407","id":263692435,"user":{"login":"ejeschke","id":1305473,"avatar_url":"https://avatars.githubusercontent.com/u/1305473?v=3","gravatar_id":"","url":"https://api.github.com/users/ejeschke","html_url":"https://github.com/ejeschke","followers_url":"https://api.github.com/users/ejeschke/followers","following_url":"https://api.github.com/users/ejeschke/following{/other_user}","gists_url":"https://api.github.com/users/ejeschke/gists{/gist_id}","starred_url":"https://api.github.com/users/ejeschke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejeschke/subscriptions","organizations_url":"https://api.github.com/users/ejeschke/orgs","repos_url":"https://api.github.com/users/ejeschke/repos","events_url":"https://api.github.com/users/ejeschke/events{/privacy}","received_events_url":"https://api.github.com/users/ejeschke/received_events","type":"User","site_admin":false},"created_at":"2016-11-29T20:42:03Z","updated_at":"2016-11-29T20:42:03Z","body":"Yes.  But I want to merge a couple of other smaller mods first.  And if #411 is actually an issue we should fix it before making a release."}}
{"actor":{"display_login":"ejeschke"}}