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

Remove ProductModel tasks from Cookbook #84

Closed
seif opened this issue Dec 17, 2012 · 2 comments
Closed

Remove ProductModel tasks from Cookbook #84

seif opened this issue Dec 17, 2012 · 2 comments
Assignees
Labels

Comments

@seif
Copy link
Member

seif commented Dec 17, 2012

The class ProductModelTasks and IProductModelTasks in the cookbook just passes calls into the repository and returns entities.

Dont see the point of doing this, if what we want to show is simply binding for a CRUD app, then we should just use the Repository in the controller directly.

@sandord
Copy link

sandord commented Jan 23, 2013

It might be wise to leave a remark that recommends delegating repository access to tasks as soon as the application (the audience is building based on the example) grows.

@cd21h cd21h self-assigned this Jan 23, 2016
@cd21h
Copy link

cd21h commented Jan 23, 2016

Also migrate project to the Samples folder.

@cd21h cd21h closed this as completed Jul 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants