Skip to content

Conversation

@sf-anilpurohit
Copy link
Collaborator

@sf-anilpurohit sf-anilpurohit commented Mar 24, 2025

What does this PR do?
In This PR we have changed Apex sample code for B2CDeliveryService to retain previously selected delivery method when we retrigger shipping calculation from buyer flow.

What issues does this PR fix or reference?
#, @@
@W-17966639

Functionality Before
<insert gif and/or summary>
Earlier we were not retaining previously selected delivery method and it was always getting clear then buyer need to select again if he change the delivery address.

Functionality After
<insert gif and/or summary>
Now we are retaining previously selected delivery method and it wont clear if buyer change the delivery address.

How to Test/Testing Effort
<insert gif and/or summary>
by placing order in checkout flow by selecting one of CDGM and changing the address we can test it.

@@ -0,0 +1,366 @@
// This must implement the sfdc_checkout.CartShippingCharges interface
Copy link
Collaborator

@wei-liu-sf wei-liu-sf Mar 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please explain this class, its purpose, its functionalities.

// This must implement the sfdc_checkout.CartShippingCharges interface
// in order to be processed by the checkout flow for the "Shipping" integration

global class B2CDeliverySampleAdvance implements sfdc_checkout.CartShippingCharges {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the legacy integration? Why do we need it?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per thread
Carly mentioned that its trust issue and changes required for all kind of stores, hence we made the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants