{"action":"closed","number":7,"pull_request":{"url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/7","id":43666219,"html_url":"https://github.com/shotgunsoftware/tk-houdini/pull/7","diff_url":"https://github.com/shotgunsoftware/tk-houdini/pull/7.diff","patch_url":"https://github.com/shotgunsoftware/tk-houdini/pull/7.patch","issue_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/7","number":7,"state":"closed","locked":false,"title":"#30708 First pass at a dynamic Shotgun shelf in Houdini","user":{"login":"josh-t","id":2604646,"avatar_url":"https://avatars.githubusercontent.com/u/2604646?v=3","gravatar_id":"","url":"https://api.github.com/users/josh-t","html_url":"https://github.com/josh-t","followers_url":"https://api.github.com/users/josh-t/followers","following_url":"https://api.github.com/users/josh-t/following{/other_user}","gists_url":"https://api.github.com/users/josh-t/gists{/gist_id}","starred_url":"https://api.github.com/users/josh-t/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/josh-t/subscriptions","organizations_url":"https://api.github.com/users/josh-t/orgs","repos_url":"https://api.github.com/users/josh-t/repos","events_url":"https://api.github.com/users/josh-t/events{/privacy}","received_events_url":"https://api.github.com/users/josh-t/received_events","type":"User","site_admin":false},"body":"Ticket #30708\r\n\r\n* Making use of houdini's hou.shelves API to build a dynamic shelf for shotgun\r\n  contexts.\r\n\r\n* Added two new settings: 'enable_sg_menu' and 'enable_sg_shelf'.\r\n\r\n* Made use of existing menu generation code so that the menu/shelf generation\r\n  shares as much logic as possible for consistency\r\n\r\n* In order to enable context switching, you need to add allowable entity types\r\n  to the tk-multi-workfile sg_entity_types setting.\r\n\r\n* The hou.shelves api means we don't have to manually generate the .xml files\r\n  the way the sg menu does. However, we still need to write to a tmp directory\r\n  so that the shelf/tool definitions don't get written to the default shelf xml\r\n  file.\r\n\r\n* Need to figure out how to programmatically add a shelf to an existing shelf\r\n  set. In this first pass, the Shotgun shelf can be added manually after\r\n  startup. The shelf will persist across sessions and update appropriately\r\n  when the context changes. However, subsequent houdini sessions run outside\r\n  of the toolkit workflow will have an empty Shotgun shelf with a message like\r\n  this: \"Cannot find the definition of the shelf 'Shotgun'.\". This is because\r\n  the default shelf xml file keeps a reference to any added shelves. Would love\r\n  to get feedback on this and input on how to make this better.\r\n\r\n* The Jump to File icon doesn't match the very similar, pre-existing and\r\n  pre-compiled, snapshot tool icons. Need to unify the look for consistency.\r\n\r\n* The 'help' text for the tools is properly populated in the tool buttons, but\r\n  does not display in the tooltip. This appears to be a bug in houdini as\r\n  other, built-in tools display the same behavior. The help urls work nicely\r\n  though!\r\n\r\n* The typically long tool names don't display very nicely in the tool button\r\n  text. Dont' think there's much we can do on that front.\r\n\r\n* Houdini automatically adds shelf tools to the viewer's tab menu. There is\r\n  some configurability here to make the sg tools available in other houdini\r\n  contexts. What would people like to see?\r\n\r\nTodo:\r\n\r\n* Add a setting which allows specified tools to be ignored and not added to the\r\n  shelf.\r\n\r\nOther:\r\n\r\n* git mv'd menu_generation.py to ui_generation.py\r\n* added vim swap file extensions to .gitignore","created_at":"2015-08-28T20:01:31Z","updated_at":"2015-09-01T01:43:02Z","closed_at":"2015-09-01T01:43:02Z","merged_at":null,"merge_commit_sha":"fb146d6ee94256eba75017418a11158cfac65d5c","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/7/commits","review_comments_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/7/comments","review_comment_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/comments{/number}","comments_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/7/comments","statuses_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/statuses/3d5f6cd5774e1c01f813b799dd886db74584ad2d","head":{"label":"shotgunsoftware:30708_hou_shelf","ref":"30708_hou_shelf","sha":"3d5f6cd5774e1c01f813b799dd886db74584ad2d","user":{"login":"shotgunsoftware","id":317894,"avatar_url":"https://avatars.githubusercontent.com/u/317894?v=3","gravatar_id":"","url":"https://api.github.com/users/shotgunsoftware","html_url":"https://github.com/shotgunsoftware","followers_url":"https://api.github.com/users/shotgunsoftware/followers","following_url":"https://api.github.com/users/shotgunsoftware/following{/other_user}","gists_url":"https://api.github.com/users/shotgunsoftware/gists{/gist_id}","starred_url":"https://api.github.com/users/shotgunsoftware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shotgunsoftware/subscriptions","organizations_url":"https://api.github.com/users/shotgunsoftware/orgs","repos_url":"https://api.github.com/users/shotgunsoftware/repos","events_url":"https://api.github.com/users/shotgunsoftware/events{/privacy}","received_events_url":"https://api.github.com/users/shotgunsoftware/received_events","type":"Organization","site_admin":false},"repo":{"id":4399203,"name":"tk-houdini","full_name":"shotgunsoftware/tk-houdini","owner":{"login":"shotgunsoftware","id":317894,"avatar_url":"https://avatars.githubusercontent.com/u/317894?v=3","gravatar_id":"","url":"https://api.github.com/users/shotgunsoftware","html_url":"https://github.com/shotgunsoftware","followers_url":"https://api.github.com/users/shotgunsoftware/followers","following_url":"https://api.github.com/users/shotgunsoftware/following{/other_user}","gists_url":"https://api.github.com/users/shotgunsoftware/gists{/gist_id}","starred_url":"https://api.github.com/users/shotgunsoftware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shotgunsoftware/subscriptions","organizations_url":"https://api.github.com/users/shotgunsoftware/orgs","repos_url":"https://api.github.com/users/shotgunsoftware/repos","events_url":"https://api.github.com/users/shotgunsoftware/events{/privacy}","received_events_url":"https://api.github.com/users/shotgunsoftware/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/shotgunsoftware/tk-houdini","description":"A Shotgun Pipeline Toolkit Engine for Houdini","fork":false,"url":"https://api.github.com/repos/shotgunsoftware/tk-houdini","forks_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/forks","keys_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/keys{/key_id}","collaborators_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/teams","hooks_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/hooks","issue_events_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/events{/number}","events_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/events","assignees_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/assignees{/user}","branches_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/branches{/branch}","tags_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/tags","blobs_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/refs{/sha}","trees_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/trees{/sha}","statuses_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/statuses/{sha}","languages_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/languages","stargazers_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/stargazers","contributors_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/contributors","subscribers_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/subscribers","subscription_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/subscription","commits_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/commits{/sha}","git_commits_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/commits{/sha}","comments_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/comments{/number}","issue_comment_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/comments{/number}","contents_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/contents/{+path}","compare_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/compare/{base}...{head}","merges_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/merges","archive_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/downloads","issues_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues{/number}","pulls_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls{/number}","milestones_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/milestones{/number}","notifications_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/labels{/name}","releases_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/releases{/id}","created_at":"2012-05-21T21:53:20Z","updated_at":"2015-01-15T16:07:23Z","pushed_at":"2015-09-01T01:42:54Z","git_url":"git://github.com/shotgunsoftware/tk-houdini.git","ssh_url":"git@github.com:shotgunsoftware/tk-houdini.git","clone_url":"https://github.com/shotgunsoftware/tk-houdini.git","svn_url":"https://github.com/shotgunsoftware/tk-houdini","homepage":"","size":56513,"stargazers_count":6,"watchers_count":6,"language":"C++","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":1,"forks":3,"open_issues":1,"watchers":6,"default_branch":"master"}},"base":{"label":"shotgunsoftware:master","ref":"master","sha":"6ae4d88fc1ec5c364848f88a57983710c6874a8d","user":{"login":"shotgunsoftware","id":317894,"avatar_url":"https://avatars.githubusercontent.com/u/317894?v=3","gravatar_id":"","url":"https://api.github.com/users/shotgunsoftware","html_url":"https://github.com/shotgunsoftware","followers_url":"https://api.github.com/users/shotgunsoftware/followers","following_url":"https://api.github.com/users/shotgunsoftware/following{/other_user}","gists_url":"https://api.github.com/users/shotgunsoftware/gists{/gist_id}","starred_url":"https://api.github.com/users/shotgunsoftware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shotgunsoftware/subscriptions","organizations_url":"https://api.github.com/users/shotgunsoftware/orgs","repos_url":"https://api.github.com/users/shotgunsoftware/repos","events_url":"https://api.github.com/users/shotgunsoftware/events{/privacy}","received_events_url":"https://api.github.com/users/shotgunsoftware/received_events","type":"Organization","site_admin":false},"repo":{"id":4399203,"name":"tk-houdini","full_name":"shotgunsoftware/tk-houdini","owner":{"login":"shotgunsoftware","id":317894,"avatar_url":"https://avatars.githubusercontent.com/u/317894?v=3","gravatar_id":"","url":"https://api.github.com/users/shotgunsoftware","html_url":"https://github.com/shotgunsoftware","followers_url":"https://api.github.com/users/shotgunsoftware/followers","following_url":"https://api.github.com/users/shotgunsoftware/following{/other_user}","gists_url":"https://api.github.com/users/shotgunsoftware/gists{/gist_id}","starred_url":"https://api.github.com/users/shotgunsoftware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shotgunsoftware/subscriptions","organizations_url":"https://api.github.com/users/shotgunsoftware/orgs","repos_url":"https://api.github.com/users/shotgunsoftware/repos","events_url":"https://api.github.com/users/shotgunsoftware/events{/privacy}","received_events_url":"https://api.github.com/users/shotgunsoftware/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/shotgunsoftware/tk-houdini","description":"A Shotgun Pipeline Toolkit Engine for Houdini","fork":false,"url":"https://api.github.com/repos/shotgunsoftware/tk-houdini","forks_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/forks","keys_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/keys{/key_id}","collaborators_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/teams","hooks_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/hooks","issue_events_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/events{/number}","events_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/events","assignees_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/assignees{/user}","branches_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/branches{/branch}","tags_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/tags","blobs_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/refs{/sha}","trees_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/trees{/sha}","statuses_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/statuses/{sha}","languages_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/languages","stargazers_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/stargazers","contributors_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/contributors","subscribers_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/subscribers","subscription_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/subscription","commits_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/commits{/sha}","git_commits_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/git/commits{/sha}","comments_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/comments{/number}","issue_comment_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/comments{/number}","contents_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/contents/{+path}","compare_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/compare/{base}...{head}","merges_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/merges","archive_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/downloads","issues_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues{/number}","pulls_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls{/number}","milestones_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/milestones{/number}","notifications_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/labels{/name}","releases_url":"https://api.github.com/repos/shotgunsoftware/tk-houdini/releases{/id}","created_at":"2012-05-21T21:53:20Z","updated_at":"2015-01-15T16:07:23Z","pushed_at":"2015-09-01T01:42:54Z","git_url":"git://github.com/shotgunsoftware/tk-houdini.git","ssh_url":"git@github.com:shotgunsoftware/tk-houdini.git","clone_url":"https://github.com/shotgunsoftware/tk-houdini.git","svn_url":"https://github.com/shotgunsoftware/tk-houdini","homepage":"","size":56513,"stargazers_count":6,"watchers_count":6,"language":"C++","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":1,"forks":3,"open_issues":1,"watchers":6,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/7"},"html":{"href":"https://github.com/shotgunsoftware/tk-houdini/pull/7"},"issue":{"href":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/7"},"comments":{"href":"https://api.github.com/repos/shotgunsoftware/tk-houdini/issues/7/comments"},"review_comments":{"href":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/7/comments"},"review_comment":{"href":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/shotgunsoftware/tk-houdini/pulls/7/commits"},"statuses":{"href":"https://api.github.com/repos/shotgunsoftware/tk-houdini/statuses/3d5f6cd5774e1c01f813b799dd886db74584ad2d"}},"merged":false,"mergeable":true,"mergeable_state":"clean","merged_by":null,"comments":0,"review_comments":0,"commits":2,"additions":520,"deletions":239,"changed_files":8}}