Skip to content

Conversation

@shivani-khandelwal
Copy link
Collaborator

Copy link

@msalian01 msalian01 left a comment

Choose a reason for hiding this comment

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

@shivani-khandelwal , the WorkItem says , the class should extend CartExtension.TaxCartCalculator instead of CartExtension.AbstractCartCalculator. are we not covering it ?

@shivani-khandelwal
Copy link
Collaborator Author

@msalian01 , that would be taken care of by sneha as part of a different item. You can refer to that discussion in the WI feed for more details.

@msalian01
Copy link

https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07EE00001Y8GM7YAN/view

Sure, With GUS feed , i got the impression that should have been already covered. Approving from my side.

@shivani-khandelwal shivani-khandelwal force-pushed the local246branch branch 5 times, most recently from fec738b to 3a9d22a Compare September 4, 2023 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants