Problem/Motivation

Integration with Commerce Shipping RC1 is currently partially broken.
For integration where a user could switch currencies, shipment in those cases contain old currency.

Proposed resolution

Add additional order processor in shipping submodule which is going to trigger recalculating shipping rates

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 commerce_shipping_recalculation_failes-3118535-3.patch22.39 KBvalic
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

valic created an issue. See original summary.

valic’s picture

valic
Primary language Croatian
Location Osijek
CreditAttribution: valic at Vallic commented
Title: Commerce shipping recalcultion failes » Commerce shipping recalculation failes
Priority: Normal » Critical
valic’s picture

valic
Primary language Croatian
Location Osijek
CreditAttribution: valic at Vallic commented
Title: Commerce shipping recalculation failes » Commerce shipping recalculation fails
Assigned: valic » Unassigned
Status: Patch (to be ported) » Needs review
FileSize
commerce_shipping_recalculation_failes-3118535-3.patch22.39 KB

Added patch. Expanded test coverage for this scenario. With few additional basic tests for current currency it self

  • valic committed f53764c on 8.x-1.x 
    Issue #3118535 by valic: Commerce shipping recalculation fails
valic’s picture

valic
Primary language Croatian
Location Osijek
CreditAttribution: valic at Vallic commented
Status: Needs review » Fixed

Tagging 1.52 to fix shipping recalculation

Status: Fixed » Closed (fixed)

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