{"action":"created","issue":{"url":"https://api.github.com/repos/facebook/flux/issues/178","labels_url":"https://api.github.com/repos/facebook/flux/issues/178/labels{/name}","comments_url":"https://api.github.com/repos/facebook/flux/issues/178/comments","events_url":"https://api.github.com/repos/facebook/flux/issues/178/events","html_url":"https://github.com/facebook/flux/pull/178","id":60889886,"number":178,"title":"I think it should be like that. Isn't it ?","user":{"login":"AlexKVal","id":847572,"avatar_url":"https://avatars.githubusercontent.com/u/847572?v=3","gravatar_id":"","url":"https://api.github.com/users/AlexKVal","html_url":"https://github.com/AlexKVal","followers_url":"https://api.github.com/users/AlexKVal/followers","following_url":"https://api.github.com/users/AlexKVal/following{/other_user}","gists_url":"https://api.github.com/users/AlexKVal/gists{/gist_id}","starred_url":"https://api.github.com/users/AlexKVal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AlexKVal/subscriptions","organizations_url":"https://api.github.com/users/AlexKVal/orgs","repos_url":"https://api.github.com/users/AlexKVal/repos","events_url":"https://api.github.com/users/AlexKVal/events{/privacy}","received_events_url":"https://api.github.com/users/AlexKVal/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-12T20:02:28Z","updated_at":"2015-03-20T16:32:13Z","closed_at":"2015-03-20T16:32:01Z","pull_request":{"url":"https://api.github.com/repos/facebook/flux/pulls/178","html_url":"https://github.com/facebook/flux/pull/178","diff_url":"https://github.com/facebook/flux/pull/178.diff","patch_url":"https://github.com/facebook/flux/pull/178.patch"},"body":"We want to trigger Store.emitChange() only if data were changed."},"comment":{"url":"https://api.github.com/repos/facebook/flux/issues/comments/84062582","html_url":"https://github.com/facebook/flux/pull/178#issuecomment-84062582","issue_url":"https://api.github.com/repos/facebook/flux/issues/178","id":84062582,"user":{"login":"fisherwebdev","id":169760,"avatar_url":"https://avatars.githubusercontent.com/u/169760?v=3","gravatar_id":"","url":"https://api.github.com/users/fisherwebdev","html_url":"https://github.com/fisherwebdev","followers_url":"https://api.github.com/users/fisherwebdev/followers","following_url":"https://api.github.com/users/fisherwebdev/following{/other_user}","gists_url":"https://api.github.com/users/fisherwebdev/gists{/gist_id}","starred_url":"https://api.github.com/users/fisherwebdev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fisherwebdev/subscriptions","organizations_url":"https://api.github.com/users/fisherwebdev/orgs","repos_url":"https://api.github.com/users/fisherwebdev/repos","events_url":"https://api.github.com/users/fisherwebdev/events{/privacy}","received_events_url":"https://api.github.com/users/fisherwebdev/received_events","type":"User","site_admin":false},"created_at":"2015-03-20T16:32:13Z","updated_at":"2015-03-20T16:32:13Z","body":"Yes, this is correct.  One could also put the `emitChange()` in the update method (some would say this is cleaner) but that adds additional side effects to that method.  I also like seeing where all the emits are happening in one place, making it easier to understand which actions cause an emit, and why, and which don't."}}