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 #354 - Reverting Redirect changes #355

Closed Navilois closed 2 years ago

Navilois commented 3 years ago

Fixes #354

Revert "All Controllers: changed every occurrence of Response.Redirect to Redirect"

This reverts commit 5fa3a900e916833e2f50f5c0b54e029ba3e079de.

erw13n commented 2 years ago

Don't revert the change that been made. Response.Redirect is causing a lot of error on event logs as reported in #112 and hotcakes forum

The implementation of Redirect was wrong, it should be "return Redirect" instead of "Redirect"

Related to #368

WillStrohl commented 2 years ago

This PR cannot be accepted as-is. It breaks numerous checkout workflows.