Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Use factory pattern for creating \ activating \ model binding a controller in ControllerActionInvoker #6249

Closed
pranavkm opened this issue May 9, 2017 · 2 comments
Closed

Use factory pattern for creating \ activating \ model binding a controller in ControllerActionInvoker #6249

pranavkm opened this issue May 9, 2017 · 2 comments
Assignees
@pranavkm
Labels
3 - Done enhancement
Milestone
2.0.0-preview2

Comments

@pranavkm
Copy link
Contributor

pranavkm commented May 9, 2017

We established a pattern of using delegates that close over state in the factory, activator and model binder for pages. This moves the onus of caching and managing state from individual components to a single piece in the ActionInvokerProvider. We should do something similar with controllers.
@pranavkm
Copy link
Contributor Author

pranavkm commented May 9, 2017

FYI @Eilon \ @rynowak

@Eilon Eilon added this to the 2.0.0-preview2 milestone May 9, 2017
@pranavkm
Copy link
Contributor Author

Fixed in c5f771d

@pranavkm pranavkm mentioned this issue May 22, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pranavkm pranavkm

Labels
3 - Done enhancement
Projects
None yet
Milestone
2.0.0-preview2
Development

No branches or pull requests
2 participants
@pranavkm @Eilon