search

HotcakesCommerce / hotcakes-commerce-core

The core of the e-commerce part of the overall solution. This is an ecommerce shopping cart solution built on top of the DNN (DotNetNuke) CMS. Anyone can do commerce online now!
https://mmmcommerce.com
MIT License
68 stars 55 forks source link

Fixes #350 - RedirectToCheckout param not working #352

Closed joshgk00 closed 3 years ago

joshgk00 commented 3 years ago

Fixes #350

Checks the RedirectResult from the HandleActionParams() and if it's something other than the Cart page, it redirects, otherwise it continues with loading the Shopping Cart

WillStrohl commented 3 years ago

Have you happened to have tested the specific redirect you were working on AND a normal checkout workflow after this update? It looks innocent enough, but the checkout steps/workflow are the most complex there are here (except maybe the marketing engine).

joshgk00 commented 3 years ago

I have. The normal checkout workflow is unaffected. I tested this with a clean install and the sample products database and haven't found any issues with the standard flow.

WillStrohl commented 3 years ago

This PR was verified by @erw13n .