Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implementation of session facade design pattern #1278 #3121

Merged
merged 6 commits into from
Jan 11, 2025

Conversation

shahdhoss
Copy link
Contributor

What problem does this PR solve?

This pull request addresses issue #1278, I implemented the Session Facade design pattern to simplify the interface for interacting with complex ecommerce functionalities. The primary goal was to demonstrate how the Session Facade can act as an intermediary, providing a simplified interface to a series of interconnected, complex operations. For more info please refer to the readme file under the session-facade module.

session-facade/pom.xml Outdated Show resolved Hide resolved
session-facade/pom.xml Outdated Show resolved Hide resolved
@shahdhoss
Copy link
Contributor Author

Thank you very much for your valuable feedback, I have reviewed the comments and made the necessary changes.

@shahdhoss shahdhoss requested a review from iluwatar December 20, 2024 12:14
iluwatar
iluwatar previously approved these changes Jan 11, 2025
@iluwatar iluwatar merged commit 8cb7c9a into iluwatar:master Jan 11, 2025
1 check passed
@iluwatar
Copy link
Owner

Looks good! Thank you for the contribution 🎉

@all-contributors please add @shahdhoss for code

Copy link
Contributor

@iluwatar

I've put up a pull request to add @shahdhoss! 🎉

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