Skip to content

refactor(almin): introduce Unit of work#224

Merged
azu merged 10 commits intomasterfrom
unit-of-work
Jul 8, 2017
Merged

refactor(almin): introduce Unit of work#224
azu merged 10 commits intomasterfrom
unit-of-work

Conversation

@azu
Copy link
Copy Markdown
Member

@azu azu commented Jul 8, 2017

Breaking Change

Store can not receive un-important payload by Store#onDsipatch.
It means that Store#onDispatch can't receive willExecutePayload.
In other words, Store#onDispatch is same with Store#receivePayload.

Unit of Work

Introduce Unit of work as a class.
It aim to implement transaction between executing UseCase.
In other word, unit of work can control updating StoreGroup.
The user can stop unintended updating StoreGroup.

Unit of work in almin life cycle.

image

close #186
follow #219

azu added 5 commits July 8, 2017 00:45
BREAKING CHANGE: Store can not receive un-important payload by Store#onDsipatch.
It means that Store#onDispatch can't receive willExecutePayload.
In other words, Store#onDispatch is same with Store#receivePayload.
@azu
Copy link
Copy Markdown
Member Author

azu commented Jul 8, 2017

We should improve performance of Unit of Work.
Currently, It contains unnessary try-catch.

@azu
Copy link
Copy Markdown
Member Author

azu commented Jul 8, 2017

We should inverstigte that the How breaking change affect exist code?

@azu azu mentioned this pull request Jul 8, 2017
@azu azu merged commit 8e08549 into master Jul 8, 2017
@azu azu deleted the unit-of-work branch July 8, 2017 15:42
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.

docs: Unit of work

1 participant