In 1.x the cart summary was a pane which embedded a view.

In 2.x we should not use a pane, because we want to allow the cart summary to be shown on multiple steps.
For example, a checkout flow that splits "Order information" into "Shipping information" and "Payment method" would want the summary on both pages.

I suggest the following:
- Add a "has_cart_summary" flag to steps.
- Add a buildCartSummary($form, $form_state) method to checkout flows.
- If the current step has a cart summary, call that method. By default it embeds a view.
- Add a template which puts the cart summary in a right column, while keeping the panes in a left column

This will allow people who want editable cart summaries to override buildCartSummary() and provide a non-views-based solution.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz created an issue. See original summary.

pashupathi nath gajawada’s picture

Assigned: Unassigned » pashupathi nath gajawada
mglaman’s picture

There hasn't been any progress posted in 25 days. Taking this over and working on it today.

mglaman’s picture

Status: Active » Needs work
FileSize
18.22 KB

Initial work. Working through the UI now, then tests.

mglaman’s picture

Status: Needs work » Needs review
FileSize
20.96 KB
71.58 KB

Patch, and screenshot. We should do a follow up to add a "summary" view mode to orders and use that in the default view.

mglaman’s picture

mglaman’s picture

Issue summary: View changes
FileSize
44.24 KB

Updated PR with default summary view mode and config.

mglaman’s picture

Title: Add a cart summary » Add an order summary

In PR discussion changed from cart summary to order summary.

  • bojanz committed ed154b7 on 8.x-2.x authored by mglaman
    Issue #2710985 by mglaman, bojanz: Add an order summary
    
bojanz’s picture

Status: Needs review » Fixed

Boom!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.