{"action":"created","issue":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3819","labels_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3819/labels{/name}","comments_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3819/comments","events_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3819/events","html_url":"https://github.com/DynamoDS/Dynamo/pull/3819","id":58868041,"number":3819,"title":"Fix regression that switching worksapce command cannot be recorded","user":{"login":"ke-yu","id":2600379,"avatar_url":"https://avatars.githubusercontent.com/u/2600379?v=3","gravatar_id":"","url":"https://api.github.com/users/ke-yu","html_url":"https://github.com/ke-yu","followers_url":"https://api.github.com/users/ke-yu/followers","following_url":"https://api.github.com/users/ke-yu/following{/other_user}","gists_url":"https://api.github.com/users/ke-yu/gists{/gist_id}","starred_url":"https://api.github.com/users/ke-yu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ke-yu/subscriptions","organizations_url":"https://api.github.com/users/ke-yu/orgs","repos_url":"https://api.github.com/users/ke-yu/repos","events_url":"https://api.github.com/users/ke-yu/events{/privacy}","received_events_url":"https://api.github.com/users/ke-yu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"Benglin","id":5086849,"avatar_url":"https://avatars.githubusercontent.com/u/5086849?v=3","gravatar_id":"","url":"https://api.github.com/users/Benglin","html_url":"https://github.com/Benglin","followers_url":"https://api.github.com/users/Benglin/followers","following_url":"https://api.github.com/users/Benglin/following{/other_user}","gists_url":"https://api.github.com/users/Benglin/gists{/gist_id}","starred_url":"https://api.github.com/users/Benglin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Benglin/subscriptions","organizations_url":"https://api.github.com/users/Benglin/orgs","repos_url":"https://api.github.com/users/Benglin/repos","events_url":"https://api.github.com/users/Benglin/events{/privacy}","received_events_url":"https://api.github.com/users/Benglin/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-02-25T08:37:48Z","updated_at":"2015-02-26T06:57:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/3819","html_url":"https://github.com/DynamoDS/Dynamo/pull/3819","diff_url":"https://github.com/DynamoDS/Dynamo/pull/3819.diff","patch_url":"https://github.com/DynamoDS/Dynamo/pull/3819.patch"},"body":"This is regression caused by my PR #3736, where `DynamoModel.CurrentWorkspace` is directly set in `DynamoViewModel.CurrentWorkspaceIndex`'s setter. Instead, we should use recordable command `SwichTabCommand` to do that.\r\n\r\n@Benglin , sorry for dragging you to the review again..."},"comment":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/comments/76131241","html_url":"https://github.com/DynamoDS/Dynamo/pull/3819#issuecomment-76131241","issue_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3819","id":76131241,"user":{"login":"Benglin","id":5086849,"avatar_url":"https://avatars.githubusercontent.com/u/5086849?v=3","gravatar_id":"","url":"https://api.github.com/users/Benglin","html_url":"https://github.com/Benglin","followers_url":"https://api.github.com/users/Benglin/followers","following_url":"https://api.github.com/users/Benglin/following{/other_user}","gists_url":"https://api.github.com/users/Benglin/gists{/gist_id}","starred_url":"https://api.github.com/users/Benglin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Benglin/subscriptions","organizations_url":"https://api.github.com/users/Benglin/orgs","repos_url":"https://api.github.com/users/Benglin/repos","events_url":"https://api.github.com/users/Benglin/events{/privacy}","received_events_url":"https://api.github.com/users/Benglin/received_events","type":"User","site_admin":false},"created_at":"2015-02-26T06:57:33Z","updated_at":"2015-02-26T06:57:33Z","body":"Okay that makes sense, sorry for the delay! LGTM @ke-yu!"}}