Skip to content

feat(almin): Support UseCase#shouldExecute#298

Merged
azu merged 4 commits intomasterfrom
almin-should-execute
Nov 25, 2017
Merged

feat(almin): Support UseCase#shouldExecute#298
azu merged 4 commits intomasterfrom
almin-should-execute

Conversation

@azu
Copy link
Copy Markdown
Member

@azu azu commented Nov 25, 2017

Allow user to implment UseCase#shouldExecute(): boolean.

fix #292

}

private willNotExecuteUseCase(args?: any[]): void {
console.log("this is will not args", args);
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

TODO

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

How to know the usecase was not exceuted for user?

@azu azu merged commit dcb561a into master Nov 25, 2017
@azu azu deleted the almin-should-execute branch November 25, 2017 11:17
This was referenced Nov 25, 2017
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.

Almin: UseCase#shouldExecute

1 participant