{"action":"created","issue":{"url":"https://api.github.com/repos/zalmoxisus/remotedev/issues/4","repository_url":"https://api.github.com/repos/zalmoxisus/remotedev","labels_url":"https://api.github.com/repos/zalmoxisus/remotedev/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/zalmoxisus/remotedev/issues/4/comments","events_url":"https://api.github.com/repos/zalmoxisus/remotedev/issues/4/events","html_url":"https://github.com/zalmoxisus/remotedev/issues/4","id":185977271,"number":4,"title":"unsubscribe in connect() is broken - handles instanceId incorrectly","user":{"login":"rob3c","id":1098077,"avatar_url":"https://avatars.githubusercontent.com/u/1098077?v=3","gravatar_id":"","url":"https://api.github.com/users/rob3c","html_url":"https://github.com/rob3c","followers_url":"https://api.github.com/users/rob3c/followers","following_url":"https://api.github.com/users/rob3c/following{/other_user}","gists_url":"https://api.github.com/users/rob3c/gists{/gist_id}","starred_url":"https://api.github.com/users/rob3c/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rob3c/subscriptions","organizations_url":"https://api.github.com/users/rob3c/orgs","repos_url":"https://api.github.com/users/rob3c/repos","events_url":"https://api.github.com/users/rob3c/events{/privacy}","received_events_url":"https://api.github.com/users/rob3c/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-28T17:26:04Z","updated_at":"2016-10-28T19:35:35Z","closed_at":null,"body":"Hi, first of all, thanks for this handy module! I've managed to get it working with Ionic 2 and @ngrx/store (via @ngrx/store-devtools) for on-device remote debugging via a custom wrapper, and the inspector is working nicely!\r\n\r\nThe only minor issue I'm running into with remotedev is the way `unsubscribe()` is handled in the `connect()` function. It mistakenly assumes that the `instanceId` is an array index instead of the configured value in the connect options. Therefore, it never actually unsubscribes anything when it's called. (I see your chrome extension code handles this fine and just needs to be ported over here.)\r\n\r\nCoincidentally, there's also a complementary bug that kind of neutralizes the problem in @ngrx/store-devtools, where they're calling `unsubscribe()` immediately upon subscription in their Observable wrapper instead of returning a lambda to call it (I'll be creating an issue over there soon as well). So this bug is temporarily allowing update actions to still be received by ngrx even though it's being called immediately after `subscribe()` lol. Unfortunately, it doesn't actually handle them correctly for time travel to work, but that's another issue to hopefully be resolved soon on their end."},"comment":{"url":"https://api.github.com/repos/zalmoxisus/remotedev/issues/comments/257009177","html_url":"https://github.com/zalmoxisus/remotedev/issues/4#issuecomment-257009177","issue_url":"https://api.github.com/repos/zalmoxisus/remotedev/issues/4","id":257009177,"user":{"login":"zalmoxisus","id":7957859,"avatar_url":"https://avatars.githubusercontent.com/u/7957859?v=3","gravatar_id":"","url":"https://api.github.com/users/zalmoxisus","html_url":"https://github.com/zalmoxisus","followers_url":"https://api.github.com/users/zalmoxisus/followers","following_url":"https://api.github.com/users/zalmoxisus/following{/other_user}","gists_url":"https://api.github.com/users/zalmoxisus/gists{/gist_id}","starred_url":"https://api.github.com/users/zalmoxisus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zalmoxisus/subscriptions","organizations_url":"https://api.github.com/users/zalmoxisus/orgs","repos_url":"https://api.github.com/users/zalmoxisus/repos","events_url":"https://api.github.com/users/zalmoxisus/events{/privacy}","received_events_url":"https://api.github.com/users/zalmoxisus/received_events","type":"User","site_admin":false},"created_at":"2016-10-28T19:35:35Z","updated_at":"2016-10-28T19:35:35Z","body":"Hi! You're right. I fixed that in `0.2.3`. Let me know if it works."}}
{"actor":{"display_login":"zalmoxisus"}}