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.

Sign up for GitHub

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

Jump to bottom

Modularization of OPEL Manager #180

Open
RedCarrottt opened this issue Apr 12, 2017 · 1 comment
Open

Modularization of OPEL Manager #180

RedCarrottt opened this issue Apr 12, 2017 · 1 comment
Assignees
@RedCarrottt
Labels
apps enhancement proposal
Milestone
Beta 2

Comments

@RedCarrottt
Copy link
Member

In Beta 1 Release 2, OPEL Manager has very complex architecture.

There is no MVC decoupling and it is the heaven of singleton objects.

The abuse of strong couplings makes the understanding and extending of OPEL Manager difficult.

old-opel-manager

Therefore, I propose to modularize OPEL Manager.

Previously, I wrote a proposal about "Modularization of App Core Manager (#132)".

Decoupling message routing part and message handling part from other code is the most important point of the issue.

As App Core Manager is, the most important point of OPEL Manager is decoupling them.

new-opel-manager

I have plan to introduce it in OPEL Beta 2(#144).
@RedCarrottt RedCarrottt added this to the Beta 2 milestone Apr 12, 2017
@RedCarrottt RedCarrottt self-assigned this Apr 12, 2017
@RedCarrottt RedCarrottt mentioned this issue Apr 12, 2017
Open
@RedCarrottt RedCarrottt mentioned this issue May 2, 2017
Merged
@RedCarrottt
Copy link
Member Author

RedCarrottt commented Jul 11, 2017 •
edited
Loading

opel-manager

@RedCarrottt RedCarrottt mentioned this issue Aug 3, 2017
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RedCarrottt RedCarrottt

Labels
apps enhancement proposal
Projects
None yet
Milestone
Beta 2
Development

No branches or pull requests
1 participant
@RedCarrottt