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

IDataProvider implementation should be mandatory to create a rule system #13

Closed
kislayverma opened this issue Sep 11, 2016 · 0 comments · Fixed by #17
Closed

IDataProvider implementation should be mandatory to create a rule system #13

kislayverma opened this issue Sep 11, 2016 · 0 comments · Fixed by #17
Assignees
@kislayverma
Labels
bug
Milestone
1.2.7

Comments

@kislayverma
Copy link
Owner

kislayverma commented Sep 11, 2016 •
edited

Since rule system data is now provided via an injected IDataProvider implementation, rulette-core should be independent of how this data is stored and retrieved.

This means that the injected implementation is mandatory, not optional as it is currently.
@kislayverma kislayverma changed the title Remove rule_system and rule_input table constructs from rulette-core IDataProvider implementation is mandatory to create a rule system Sep 11, 2016
@kislayverma kislayverma changed the title IDataProvider implementation is mandatory to create a rule system IDataProvider implementation should be mandatory to create a rule system Sep 11, 2016
@kislayverma kislayverma added this to the 1.2.7 milestone Sep 11, 2016
@kislayverma kislayverma self-assigned this Sep 11, 2016
@kislayverma kislayverma mentioned this issue Sep 11, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kislayverma kislayverma

Labels
bug
Projects
None yet
Milestone
1.2.7
Development

Successfully merging a pull request may close this issue.

Made IDataProvider implementation mandatory in creating RuleSystem instance kislayverma/Rulette
1 participant
@kislayverma