Skip to content

After payment, need to delete the shopping cart in session and cookies#568

Merged
sarahelsaig merged 9 commits into
OrchardCMS:mainfrom
infofromca:DeleteShoppingCart
Apr 24, 2025
Merged

After payment, need to delete the shopping cart in session and cookies#568
sarahelsaig merged 9 commits into
OrchardCMS:mainfrom
infofromca:DeleteShoppingCart

Conversation

@infofromca
Copy link
Copy Markdown
Contributor

Fix #543

@infofromca
Copy link
Copy Markdown
Contributor Author

@sarahelsaig please review it

Comment on lines +11 to +10
/// When deriving a custom implementation, please inherit from <see cref="ShoppingCartPersistenceBase"/> to retain event
/// When deriving a custom implementation, please inherit from ShoppingCartPersistenceBase to retain event
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What's the reason of this change?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You've marked this as resolved without answering the question. Please resolve conversations, that's for the reviewer. (Automatically resolving via "commit suggestion" is fine.)

Comment thread src/Modules/OrchardCore.Commerce/Services/ShoppingCartPersistenceBase.cs Outdated
Comment thread src/Modules/OrchardCore.Commerce/Services/SessionShoppingCartPersistence.cs Outdated
Comment thread test/OrchardCore.Commerce.Tests/Fakes/FakeCartStorage.cs Outdated
@infofromca infofromca requested a review from sarahelsaig April 15, 2025 05:47
Comment thread src/Modules/OrchardCore.Commerce/Services/ShoppingCartPersistenceBase.cs Outdated
Comment on lines +11 to +10
/// When deriving a custom implementation, please inherit from <see cref="ShoppingCartPersistenceBase"/> to retain event
/// When deriving a custom implementation, please inherit from ShoppingCartPersistenceBase to retain event
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You've marked this as resolved without answering the question. Please resolve conversations, that's for the reviewer. (Automatically resolving via "commit suggestion" is fine.)

@infofromca infofromca requested a review from sarahelsaig April 15, 2025 22:10
@sarahelsaig
Copy link
Copy Markdown
Contributor

@infofromca This is still not addressed: #568 (comment)

@infofromca
Copy link
Copy Markdown
Contributor Author

Hi @sarahelsaig did you see Because of the separation, other project need to reference this interface. like IShoppingCartHelpers,
and also fix partially #429 in the dialogue? I do not know why it has showed pending.

@infofromca
Copy link
Copy Markdown
Contributor Author

@sarahelsaig The most importance is that PaymentService need IShoppingCartPersistence, so move it to Abstraction project, otherwise it will have circle ref.
For removing , because ShoppingCartPersistenceBase is not in Abstraction project

@sarahelsaig
Copy link
Copy Markdown
Contributor

Hi @sarahelsaig did you see Because of the separation, other project need to reference this interface. like IShoppingCartHelpers, and also fix partially #429 in the dialogue? I do not know why it has showed pending.

I haven't seen any comments from you before this. If they are stuck in "pending", that's because GitHub thinks you are writing a review. You can submit it as a comment like this:
image

@sarahelsaig sarahelsaig merged commit d51e39c into OrchardCMS:main Apr 24, 2025
6 checks passed
@infofromca infofromca deleted the DeleteShoppingCart branch April 24, 2025 14:34
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.

After payment, need to delete the shopping cart in session and cookies (OCC-330)

2 participants