{"action":"created","issue":{"url":"https://api.github.com/repos/BigNoid/script.skinshortcuts/issues/26","labels_url":"https://api.github.com/repos/BigNoid/script.skinshortcuts/issues/26/labels{/name}","comments_url":"https://api.github.com/repos/BigNoid/script.skinshortcuts/issues/26/comments","events_url":"https://api.github.com/repos/BigNoid/script.skinshortcuts/issues/26/events","html_url":"https://github.com/BigNoid/script.skinshortcuts/pull/26","id":95067074,"number":26,"title":"Rewrite of code to generate 'list' property for shortcuts","user":{"login":"Ignoble61","id":12768752,"avatar_url":"https://avatars.githubusercontent.com/u/12768752?v=3","gravatar_id":"","url":"https://api.github.com/users/Ignoble61","html_url":"https://github.com/Ignoble61","followers_url":"https://api.github.com/users/Ignoble61/followers","following_url":"https://api.github.com/users/Ignoble61/following{/other_user}","gists_url":"https://api.github.com/users/Ignoble61/gists{/gist_id}","starred_url":"https://api.github.com/users/Ignoble61/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ignoble61/subscriptions","organizations_url":"https://api.github.com/users/Ignoble61/orgs","repos_url":"https://api.github.com/users/Ignoble61/repos","events_url":"https://api.github.com/users/Ignoble61/events{/privacy}","received_events_url":"https://api.github.com/users/Ignoble61/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-07-14T23:33:25Z","updated_at":"2015-07-15T07:51:15Z","closed_at":"2015-07-15T05:55:24Z","pull_request":{"url":"https://api.github.com/repos/BigNoid/script.skinshortcuts/pulls/26","html_url":"https://github.com/BigNoid/script.skinshortcuts/pull/26","diff_url":"https://github.com/BigNoid/script.skinshortcuts/pull/26.diff","patch_url":"https://github.com/BigNoid/script.skinshortcuts/pull/26.patch"},"body":"Changes to code to generate 'list' property - supercedes #16, as discussed @14b215dd08\r\n\r\n - Cleaner code, splitting on commas to grab appropriate part of the shortcut\r\n - Only works for ActivateWindow shortcuts (other shortcuts will return as-is - let me know if there are any other shortcut types that should provide a specific 'list' property)\r\n - Moved to datafunctions.py (because I intend to use this for updated widget selection code, and because xmlfunctions has become out of control and code needs moving out of there!)\r\n\r\n@marcelveldt - this is your feature, so can you please review to ensure I haven't broken the use case you originally added this for.\r\n@HitcherUK and @Nessus85100 - both of you reported issues with the original code, so can you please test to ensure the code fixes any issues / does not introduce any new issues / there's no other obvious use-cases I should add whilst I'm working on the code.\r\n\r\nCheers."},"comment":{"url":"https://api.github.com/repos/BigNoid/script.skinshortcuts/issues/comments/121520667","html_url":"https://github.com/BigNoid/script.skinshortcuts/pull/26#issuecomment-121520667","issue_url":"https://api.github.com/repos/BigNoid/script.skinshortcuts/issues/26","id":121520667,"user":{"login":"Ignoble61","id":12768752,"avatar_url":"https://avatars.githubusercontent.com/u/12768752?v=3","gravatar_id":"","url":"https://api.github.com/users/Ignoble61","html_url":"https://github.com/Ignoble61","followers_url":"https://api.github.com/users/Ignoble61/followers","following_url":"https://api.github.com/users/Ignoble61/following{/other_user}","gists_url":"https://api.github.com/users/Ignoble61/gists{/gist_id}","starred_url":"https://api.github.com/users/Ignoble61/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ignoble61/subscriptions","organizations_url":"https://api.github.com/users/Ignoble61/orgs","repos_url":"https://api.github.com/users/Ignoble61/repos","events_url":"https://api.github.com/users/Ignoble61/events{/privacy}","received_events_url":"https://api.github.com/users/Ignoble61/received_events","type":"User","site_admin":false},"created_at":"2015-07-15T07:51:15Z","updated_at":"2015-07-15T07:51:15Z","body":"Thanks for testing + merging :+1: "}}