GitHub PushEvent at 2014-06-26 04:06:28
Repository: spree
Actor: radar
Payload: {"shas":[["5483c0210be8b966fa103b444432f4885dcf9d16","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","More reasonable order payment state logic\n\nFor simplicity sake. Admin users can still go to the orders specific\npayments section for more detailed info on payments","Washington Luiz",false],["ba2e46bbd6860defceb02a663a2cae250f026180","5e1c0d6f8a0b550386d5f8342017b9898399fe26@gmail.com","Remove Spree::Stock::OrderCounter\n\nThis is unused as of 15122ec, when the Stock::RemainingPacker was\nremoved.\n\nFixes #4659","John Hawthorn",false],["c50c60ab9b4389227bd7eb5c59a32ed6ff29fd5c","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Revert \"Refactor Shipments API & JS.\"\n\nThis was acciddentally committed. Original is 5224bf294163b168fd1ee5cc760187ec9d83c26f\n\nThis reverts commit 715311656d14f5ff7cfe9e837f8998721496c8f4.","Washington Luiz",false],["1d248078e02aa3bb7b176a57723184c9c9f84ffe","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Revert \"Add shipments API back under orders\"\n\nThis reverts commit ab17831db1f1f2c8a7fca78dda97e9f7e13dbeef.","Washington Luiz",false],["1f3e4411832311b40d58a38257076d9ece21d722","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Drop api/checkouts#create and api/checkouts#show\n\nThey already exists in api/orders","Washington Luiz",false],["77c3edfa6a65da82db01c542f004b32bb2fc4815","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Move find_order to before_filter in api/orders","Washington Luiz",false],["ed2ed76c231354f8c642f01ee09d1017ed52a4fc","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Drop order.update! in api/orders#empty\n\nAll work to empty an order should probably stay in Order#empty no need\nto evil update!","Washington Luiz",false],["473d0d2f837d33556dacb98b2b0002f89d59c531","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","OrderContents#add already saves a line item","Washington Luiz",false],["80043271dca98aab7329fc69a26cd61634e4db36","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Use OrderContents#update_cart in api/orders to update it\n\nRather than duplicating stuff already provided by spree_core","Washington Luiz",false],["9ce62a2694e83ac095ca86c1f3877e1f3f4efe06","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Drop useless api/orders#next! private method","Washington Luiz",false],["87023429b8548d5ff48bdde5774948fddb1ff05c","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Drop line items stuff from api/checkouts controller","Washington Luiz",false],["a6c27447b30fed2b6e1b890c72cd4e3005ac090e","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Drop duplicated call in api/line_items#update\n\nOrderContents#update_cart already call Order#ensure_updated_shipments","Washington Luiz",false],["d0c6166d686f014689ab39a19660c0742d4beed1","a1a2cafa0fdb38c0c1ba9a8684bc380a939751b5@BurlingtonWebApps.com","Updating price must touch variants to bust cache.","Jeff Dutil",false],["d3b9346eac0027bb34a14fcd31ffa7d1d83a5902","8d4b5bb11f4cd085e4c1e46b2914d44e09d21a01@gmail.com","Update item count and totals on Order#merge!\n\nFixes #4610","Washington Luiz",false],["a4f0ed80262827e7753adc6a15451e4c8a5ae40b","a1a2cafa0fdb38c0c1ba9a8684bc380a939751b5@BurlingtonWebApps.com","Capybara helpers should be available in extensions by default.","Jeff Dutil",false],["90ae136826235da2ac590cbfc622ca7895666c15","43ac67fe1e2fd48ddd16bed18585c27721642137@gmail.com","[frontend] Load taxon if params[:taxon] exists in ProductsController\n\nFixes a small bug where searching *twice* for the same term results in the taxon 'scope' being forgotten","Ryan Bigg",false],["c9f4d043a5de2984dc4ba2496bf60567c3c2a192","d1c700ac71de81912147baa865592690b5ab1918@gmail.com","Speed up admin/shipping_methods/_form with local var\n\nI saw [almost 200](http://en.wikipedia.org/wiki/List_of_sovereign_states) calls  to `Spree::Zone Exists (0.3ms)` and thought there must be a better way.\n\nHere we put the `@shipping_method.zones` into a local variable so we can hit that instead of the db.\n\nNotes:\n\n* The `to_a` method is necessary to force Rails to evaluate the `zones` scope. Otherwise we still get hundreds of db calls.\n* If we want to, we could always replace the array of `Zones` with an array of `ids` -- replacing `to_a` with `.pluck(:id)` and having the checkbox tag read `include?(zone.id)` instead of `include?(zone)`. However I'm not sure this additional change is necessary.\n\nFixes #4535","brchristian",false],["344ca1d2c6a0b2ae1cf1c041d12e02628d208b8d","d1c700ac71de81912147baa865592690b5ab1918@gmail.com","No longer use insufficient_stock_lines in view\n\nFixes #4665\nFixes #4664","brchristian",false],["3ea8a759943d9d67bfe54b9a531cad73e8edc9c6","d1c700ac71de81912147baa865592690b5ab1918@gmail.com","No longer caching insufficient_stock_lines\n\nFixes #4664\nFixes #4666","brchristian",false],["c76ac47c619e6419954b043f47f73e2d762196ef","5e1c0d6f8a0b550386d5f8342017b9898399fe26@gmail.com","Stock::Quantifier must take a variant, not an id\n\n@brchristian pointed out that the previous resolve_variant_id logic was\nbroken when a variant had been deleted.\n\nThis removes the resolve_variant_id private method from Quantifier, and\nreuqires that a variant object be passed in. Finding associated variants\nshould be the resposibility of the caller, and the code to load a\nvariant unscoped need not be here.\n\nFixes #4660","John Hawthorn",false]],"size":139,"ref":"refs/heads/master","head":"fd8960a1cf494718ceaa8ed567a4927255a11da0"}