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

Port Architecture for module Development so Can use both inbuild custom feature development as well external integration #508

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lalittmohan
Copy link

@lalittmohan lalittmohan commented Apr 22, 2022

Problem

we have various integration so need different module to add functionality and we have some functionality inbuilt so Need to create single module to add all possible integration domain wise like Reviews, Product Management , Carriers, Payment, Order, etc/

Example: Customer need some third party review services from external source like google or trustpilot so need custom module to integrate & API Adapter to compatible with existing review system in-built.

  • Existing system doesn't support for DDD by use-case. As every customer have different use-case for same features/functionality

Solution

Main focus is to change module structure to ports architecture so module is more testable by use-case & domain oriented so we can use single module to add all possible integration for payment OR shipping in plug-n-play manner.

  • testable by use-case( can be test like unit test )
  • DDD (domain driven development)
  • Adapter test
  • Loosely couple

Requested Reviewers

@lalittmohan lalittmohan reopened this Apr 22, 2022
@lalittmohan lalittmohan reopened this Apr 22, 2022
@lalittmohan lalittmohan changed the title Port Architecture for module Development so Can use both inbuild feature as well external integration Port Architecture for module Development so Can use both inbuild custom feature development as well external integration Apr 22, 2022
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.

1 participant