CommerceShippingSubscriber is currently responsible not only for creating shipment, but also for setting order email and amending payment price.

Those two should be ReturnFromVippsExpressSubscriber::createBillingProfile

CommentFileSizeAuthor
#2 3082939-2.patch7.64 KBzaporylie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zaporylie created an issue. See original summary.

zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie for Ny Media AS commented
Status: Active » Needs review
FileSize
3082939-2.patch7.64 KB

And the patch.

  • zaporylie authored b12bf63 on 8.x-4.x 
    Issue #3082939 by zaporylie: CommerceShippingSubscriber should only set...
zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie for Ny Media AS commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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