{"action":"created","issue":{"url":"https://api.github.com/repos/shotgunsoftware/tk-desktop/issues/28","labels_url":"https://api.github.com/repos/shotgunsoftware/tk-desktop/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/shotgunsoftware/tk-desktop/issues/28/comments","events_url":"https://api.github.com/repos/shotgunsoftware/tk-desktop/issues/28/events","html_url":"https://github.com/shotgunsoftware/tk-desktop/pull/28","id":113911666,"number":28,"title":"#31557 Friendlier Errors on Empty OS Path","user":{"login":"JesseEmond","id":1843555,"avatar_url":"https://avatars.githubusercontent.com/u/1843555?v=3","gravatar_id":"","url":"https://api.github.com/users/JesseEmond","html_url":"https://github.com/JesseEmond","followers_url":"https://api.github.com/users/JesseEmond/followers","following_url":"https://api.github.com/users/JesseEmond/following{/other_user}","gists_url":"https://api.github.com/users/JesseEmond/gists{/gist_id}","starred_url":"https://api.github.com/users/JesseEmond/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JesseEmond/subscriptions","organizations_url":"https://api.github.com/users/JesseEmond/orgs","repos_url":"https://api.github.com/users/JesseEmond/repos","events_url":"https://api.github.com/users/JesseEmond/events{/privacy}","received_events_url":"https://api.github.com/users/JesseEmond/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"kporangehat","id":3391,"avatar_url":"https://avatars.githubusercontent.com/u/3391?v=3","gravatar_id":"","url":"https://api.github.com/users/kporangehat","html_url":"https://github.com/kporangehat","followers_url":"https://api.github.com/users/kporangehat/followers","following_url":"https://api.github.com/users/kporangehat/following{/other_user}","gists_url":"https://api.github.com/users/kporangehat/gists{/gist_id}","starred_url":"https://api.github.com/users/kporangehat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kporangehat/subscriptions","organizations_url":"https://api.github.com/users/kporangehat/orgs","repos_url":"https://api.github.com/users/kporangehat/repos","events_url":"https://api.github.com/users/kporangehat/events{/privacy}","received_events_url":"https://api.github.com/users/kporangehat/received_events","type":"User","site_admin":false},"milestone":null,"comments":6,"created_at":"2015-10-28T20:09:56Z","updated_at":"2015-11-06T19:14:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/shotgunsoftware/tk-desktop/pulls/28","html_url":"https://github.com/shotgunsoftware/tk-desktop/pull/28","diff_url":"https://github.com/shotgunsoftware/tk-desktop/pull/28.diff","patch_url":"https://github.com/shotgunsoftware/tk-desktop/pull/28.patch"},"body":"Two small fixes with this PR:\r\n\r\n- Friendlier error message if desktop tries to open a pipeline config that doesn't have a path set for the current OS.\r\n- Allow to change pipeline config even if the current pipeline config fails because of an empty OS path (useful as a dev if e.g. Windows is not set on primary, but it is set on an other pipeline config for testing purposes). This is done by moving the context menu initialization before the code that reads the config.\r\n\r\nHere's how master currently handles it:\r\n\r\n![screen shot 2015-10-27 at 3 14 56 pm](https://cloud.githubusercontent.com/assets/14185921/10769757/15c41800-7cbe-11e5-96e2-303f6fa4564f.png)\r\n\r\nTwo things to notice here:\r\n\r\n- error message is too technical\r\n- can't change primary config\r\n\r\nWith this fix:\r\n![screen shot 2015-10-27 at 3 15 49 pm](https://cloud.githubusercontent.com/assets/14185921/10769782/31697e88-7cbe-11e5-9383-221c03fed34d.png)\r\n"},"comment":{"url":"https://api.github.com/repos/shotgunsoftware/tk-desktop/issues/comments/154504818","html_url":"https://github.com/shotgunsoftware/tk-desktop/pull/28#issuecomment-154504818","issue_url":"https://api.github.com/repos/shotgunsoftware/tk-desktop/issues/28","id":154504818,"user":{"login":"JesseEmond","id":1843555,"avatar_url":"https://avatars.githubusercontent.com/u/1843555?v=3","gravatar_id":"","url":"https://api.github.com/users/JesseEmond","html_url":"https://github.com/JesseEmond","followers_url":"https://api.github.com/users/JesseEmond/followers","following_url":"https://api.github.com/users/JesseEmond/following{/other_user}","gists_url":"https://api.github.com/users/JesseEmond/gists{/gist_id}","starred_url":"https://api.github.com/users/JesseEmond/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JesseEmond/subscriptions","organizations_url":"https://api.github.com/users/JesseEmond/orgs","repos_url":"https://api.github.com/users/JesseEmond/repos","events_url":"https://api.github.com/users/JesseEmond/events{/privacy}","received_events_url":"https://api.github.com/users/JesseEmond/received_events","type":"User","site_admin":false},"created_at":"2015-11-06T19:14:36Z","updated_at":"2015-11-06T19:14:36Z","body":"@kporangehat that actually sounded like a *very* good idea, although the errors in this case use `tk-framework-qtwidgets`' overlay for the error text, which is simply **painting** the text. This prevents us from adding rich text (e.g. links), and putting the raw URL there doesn't help either because it can't be selected or copied... I tried it and it was more frustrating than anything to have an unclickable/unselectable link there.\r\n\r\nI have create an internal ticket (# 33179) to allow `tk-framework-qtwidgets`'overlay to handle widgets.\r\n\r\nI agreed with your feedback and I have changed the error messages so that both cases advise the user to go to the website:\r\n![screen shot 2015-11-06 at 2 02 12 pm](https://cloud.githubusercontent.com/assets/1843555/11006188/5b42b90c-8490-11e5-8388-c97ee168cd3f.png)\r\n![screen shot 2015-11-06 at 2 03 56 pm](https://cloud.githubusercontent.com/assets/1843555/11006189/5d4dfe14-8490-11e5-80fa-fb5194f1aa7b.png)\r\n\r\nUnfortunate that we can't put a link in there yet, but would it be good enough for now?"}}