Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Shipping Estimator loads NTFD when SERT doesn't have URI #68

Closed
steveosoule opened this issue Jun 6, 2019 · 0 comments · Fixed by #79
Closed

Shipping Estimator loads NTFD when SERT doesn't have URI #68

steveosoule opened this issue Jun 6, 2019 · 0 comments · Fixed by #79
Assignees
Milestone

Comments

@steveosoule
Copy link

steveosoule commented Jun 6, 2019

Expected Behavior

Using the Shipping Estimate should load the SERT page and return rates; even if SERT doesn't have a canonical URI.

Current Behavior

Shipping Estimator loads the NTFD page (when the SERT page doesn't have a URI)

Attempted Fixes

Update https://github.com/mivaecommerce/readytheme-shadows/blob/master/editable_templates/shadows/mm5/BASK-shipestimate-item.htm#L4 from method="get" to method="post"

Steps to Reproduce (for bugs)

  1. Find a store that doesn't have a URI on the SERT page (some legacy Miva stores), or just delete the SERT page's URIs for replication sake.
  2. Add a product to your basket: https://ssoule.mivamerchantdev.com/store/sample-category/miva-test_master-product.html
  3. Navigate to the BASK page: https://ssoule.mivamerchantdev.com/store/basket-contents.html
  4. Click the "Estimate" button.
  5. Fill out the fields in the modal.
  6. Click the "Calculate" button
  7. It loads the NTFD page instead of the SERT page
@influxweb influxweb self-assigned this Jun 12, 2019
@influxweb influxweb added this to the v1.0.6 milestone Jun 12, 2019
influxweb added a commit that referenced this issue Oct 18, 2019
This maintenance release addresses all issues contained in the v1.0.6 milestone.
https://github.com/mivaecommerce/readytheme-shadows/milestone/7?closed=1

This closes #68, closes #69, closes #70, closes #71, closes #72, closes #73, closes #74, closes #75, closes #76, closes #77, and closes #78
@influxweb influxweb mentioned this issue Oct 18, 2019
influxweb added a commit that referenced this issue Oct 18, 2019
This maintenance release addresses all issues contained in the v1.0.6 milestone.
https://github.com/mivaecommerce/readytheme-shadows/milestone/7?closed=1

This closes #68, closes #69, closes #70, closes #71, closes #72, closes #73, closes #74, closes #75, closes #76, closes #77, and closes #78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants