Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
With #2896815: Replace the order summary view with a Twig template we added a Twig template to handle displaying the checkout sidebar summary instead of a view display.
EXCEPT, we never removed the view
value from the default config! 😱
order_summary:
view: commerce_checkout_order_summary
step: _sidebar
weight: 5
We need to set the view
property to a null value. Probably by just removing it.
Comment | File | Size | Author |
---|---|---|---|
#6 | 3087512-unused-view-sidebar-summary-2.patch | 593 bytes | amjad1233 |
| |||
#3 | 3087512-unused-view-sidebar-summary.patch | 575 bytes | amjad1233 |
|
Comments
Comment #2
amjad1233Comment #3
amjad1233Comment #4
bojanz CreditAttribution: bojanz at Centarro commented#3 seems to have been generated by editing the YML file by hand, which produces output that doesn't match what would be if you edited the checkout flow and then re-exported config.
Comment #5
amjad1233Good pick. Cool I will do that. I apologise as I am new to contribution space.
Comment #6
amjad1233@bojanz When I exported the config, It came with other stuff as well which I omitted thinking to match the default config of the module.
i.e., I removed
Is that seems right ?
Comment #7
mglamanI think it's OK to omit the other added config (the default config on install is just out of date in general.) This issue is about fixing an oversight from the initial issue.
What do you think, @bojanz?
I thought I saw an issue about general "refresh all of our existing configs", but I can't find it. And that wouldn't fix this issue.
Comment #9
bojanz CreditAttribution: bojanz at Centarro commented#6 looks good. Committed. Thanks!
We'll want to update the whole export in another issue. We've already re-exported a lot of our config in 2.14, but there's always more.
Comment #10
amjad1233Cheers Guys thanks for that.