{"action":"created","issue":{"url":"https://api.github.com/repos/paypal/react-engine/issues/169","repository_url":"https://api.github.com/repos/paypal/react-engine","labels_url":"https://api.github.com/repos/paypal/react-engine/issues/169/labels{/name}","comments_url":"https://api.github.com/repos/paypal/react-engine/issues/169/comments","events_url":"https://api.github.com/repos/paypal/react-engine/issues/169/events","html_url":"https://github.com/paypal/react-engine/pull/169","id":168447924,"number":169,"title":"Change Client From LoDash Merge to LoDash Assign","user":{"login":"JustinMGaiam","id":16195149,"avatar_url":"https://avatars.githubusercontent.com/u/16195149?v=3","gravatar_id":"","url":"https://api.github.com/users/JustinMGaiam","html_url":"https://github.com/JustinMGaiam","followers_url":"https://api.github.com/users/JustinMGaiam/followers","following_url":"https://api.github.com/users/JustinMGaiam/following{/other_user}","gists_url":"https://api.github.com/users/JustinMGaiam/gists{/gist_id}","starred_url":"https://api.github.com/users/JustinMGaiam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JustinMGaiam/subscriptions","organizations_url":"https://api.github.com/users/JustinMGaiam/orgs","repos_url":"https://api.github.com/users/JustinMGaiam/repos","events_url":"https://api.github.com/users/JustinMGaiam/events{/privacy}","received_events_url":"https://api.github.com/users/JustinMGaiam/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-07-30T04:10:04Z","updated_at":"2016-07-30T13:52:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/paypal/react-engine/pulls/169","html_url":"https://github.com/paypal/react-engine/pull/169","diff_url":"https://github.com/paypal/react-engine/pull/169.diff","patch_url":"https://github.com/paypal/react-engine/pull/169.patch"},"body":"- Changing lodash.merge to lodash.assign for speed and to correct memory usage issues in lib/client.js for issue 166. https://github.com/paypal/react-engine/issues/166"},"comment":{"url":"https://api.github.com/repos/paypal/react-engine/issues/comments/236366367","html_url":"https://github.com/paypal/react-engine/pull/169#issuecomment-236366367","issue_url":"https://api.github.com/repos/paypal/react-engine/issues/169","id":236366367,"user":{"login":"JustinMGaiam","id":16195149,"avatar_url":"https://avatars.githubusercontent.com/u/16195149?v=3","gravatar_id":"","url":"https://api.github.com/users/JustinMGaiam","html_url":"https://github.com/JustinMGaiam","followers_url":"https://api.github.com/users/JustinMGaiam/followers","following_url":"https://api.github.com/users/JustinMGaiam/following{/other_user}","gists_url":"https://api.github.com/users/JustinMGaiam/gists{/gist_id}","starred_url":"https://api.github.com/users/JustinMGaiam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JustinMGaiam/subscriptions","organizations_url":"https://api.github.com/users/JustinMGaiam/orgs","repos_url":"https://api.github.com/users/JustinMGaiam/repos","events_url":"https://api.github.com/users/JustinMGaiam/events{/privacy}","received_events_url":"https://api.github.com/users/JustinMGaiam/received_events","type":"User","site_admin":false},"created_at":"2016-07-30T13:52:13Z","updated_at":"2016-07-30T13:52:13Z","body":"\r\nThis greatly increase the router change speed as well. When using Redux and ImmutableJS data can get very expensive in terms of doing the deep copy's which is why ImmutableJS provides its own merge method.  This replaces lodash's merge with assign so a shallow copy is performed.\r\n\r\nIt seems to me that a merge is unnecessary, if I understand what the client is doing correctly, when creating the router component.  Unless you want props with the same name on both the router component and child component to combine together into a single prop as it does with merge, assign will just favor the router prop of the same name over the child component prop.\r\n\r\nIf this needs to be a more dynamic strategy I am curious what this is actually designed to do, but for now this works in our app and greatly improves performance which currently is unusable in both FireFox and Internet Explorer.  We did not run into issue until our store data started to grow over time."}}
{"actor":{"display_login":"JustinMGaiam"}}