{"action":"created","issue":{"url":"https://api.github.com/repos/bonobos/solidus/issues/3","labels_url":"https://api.github.com/repos/bonobos/solidus/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/bonobos/solidus/issues/3/comments","events_url":"https://api.github.com/repos/bonobos/solidus/issues/3/events","html_url":"https://github.com/bonobos/solidus/pull/3","id":124517799,"number":3,"title":"preserve shipping method when shipments change","user":{"login":"gvaughn","id":10173,"avatar_url":"https://avatars.githubusercontent.com/u/10173?v=3","gravatar_id":"","url":"https://api.github.com/users/gvaughn","html_url":"https://github.com/gvaughn","followers_url":"https://api.github.com/users/gvaughn/followers","following_url":"https://api.github.com/users/gvaughn/following{/other_user}","gists_url":"https://api.github.com/users/gvaughn/gists{/gist_id}","starred_url":"https://api.github.com/users/gvaughn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gvaughn/subscriptions","organizations_url":"https://api.github.com/users/gvaughn/orgs","repos_url":"https://api.github.com/users/gvaughn/repos","events_url":"https://api.github.com/users/gvaughn/events{/privacy}","received_events_url":"https://api.github.com/users/gvaughn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-12-31T23:49:17Z","updated_at":"2016-01-07T22:15:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bonobos/solidus/pulls/3","html_url":"https://github.com/bonobos/solidus/pull/3","diff_url":"https://github.com/bonobos/solidus/pull/3.diff","patch_url":"https://github.com/bonobos/solidus/pull/3.patch"},"body":"The main impetus of this is Tulip in which the guides have a much more\r\nnon-linear approach to creating an order, but the same problem exists\r\nthrough the web client too.\r\n\r\nOther approaches have been tried, but lets try this within the\r\nEstimator itself. An extra side-effect of this is that we don't\r\ndelete shipments in Order#ensure_updated_shipments so that's poorly\r\nnamed now, but it also raises a question of why it's called when it\r\nis. This means a shipment will stick with an order if created before we\r\nreach the \"delivery\" state of the order. Upon transition into \"delivery\"\r\nthen #created_proposed_shipments gets called to ensure they're correct.\r\nThe new logic in the Estimator now will re-select any prior shipping\r\nmethod/rate selection the customer had previously made."},"comment":{"url":"https://api.github.com/repos/bonobos/solidus/issues/comments/169823701","html_url":"https://github.com/bonobos/solidus/pull/3#issuecomment-169823701","issue_url":"https://api.github.com/repos/bonobos/solidus/issues/3","id":169823701,"user":{"login":"gvaughn","id":10173,"avatar_url":"https://avatars.githubusercontent.com/u/10173?v=3","gravatar_id":"","url":"https://api.github.com/users/gvaughn","html_url":"https://github.com/gvaughn","followers_url":"https://api.github.com/users/gvaughn/followers","following_url":"https://api.github.com/users/gvaughn/following{/other_user}","gists_url":"https://api.github.com/users/gvaughn/gists{/gist_id}","starred_url":"https://api.github.com/users/gvaughn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gvaughn/subscriptions","organizations_url":"https://api.github.com/users/gvaughn/orgs","repos_url":"https://api.github.com/users/gvaughn/repos","events_url":"https://api.github.com/users/gvaughn/events{/privacy}","received_events_url":"https://api.github.com/users/gvaughn/received_events","type":"User","site_admin":false},"created_at":"2016-01-07T22:15:50Z","updated_at":"2016-01-07T22:15:50Z","body":"This is ready for final review. (I don't see a useful label to apply nor how to add one)\r\n\r\ncore and api specs pass cleanly. backend and frontend have sporadic failures, which also happens on the parent branch. We will circle back to get that code into CI and fixup failing specs as a separate task."}}