Is there something extra I need to do combine carts?

Thank you for this #7 patch!

Using this module with patch #7, I am noticing on our site, when a user adds a product to their cart, it generates a status message for all the products in the cart, including products that were already in the cart and not added in that request.

@bob.hinrichs

Tested this on me, can you confirm that the error always appears as soon as the order type changes?

Hi @ruvus, this seems to happen anytime an item is added to the cart. It works fine in the cart, but we just get a ton of status messages on the page, one for each item in the cart

First, I would also add this file:

commerce_combine_carts.settings:
  title: 'Combine Carts Settings'
  description: 'Combining different order types.'
  route_name: commerce_combine_carts.settings
  parent: 'commerce_order.configuration'
  weight: 999

as commerce_combine_carts.links.menu.yml. This makes the settings form appear as a link in the normal Commerce settings dashboard.

Second, although I found the carts tobe combined, I'm not sure the shipment methods are handled all right. If the two carts have different shipping methods and rates — and they probably have, that's the main reason for them in the first place — how is that supposed to be handled?

Hi @ruvus,
Thank you for the patch. I would suggest to change the permission level to 'access commerce administration pages' on the routing.

Using this module with patch #7, I am noticing on our site, when a user adds a product to their cart, it generates separate status messages for all the products in the cart, including products that were already in the cart and not added in that request, so that the user sees multiple status messages on the screen, only one of which is true/relevant.

It might be worth adding a separate issue about this.

Updating priority bc the module not doing what was designed to do is critical.