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

Simplify MvcOptions #2269

Closed
rynowak opened this issue Mar 25, 2015 · 0 comments
Closed

Simplify MvcOptions #2269

rynowak opened this issue Mar 25, 2015 · 0 comments

Comments

@rynowak
Copy link
Member

rynowak commented Mar 25, 2015

From API review:

We want to move away from OptionDescriptor<T>. The desire here is to simplify the lifecycle for the various components that go into options - in general they should be instances/singletons.

There may be some cases where the non-singleton makes sense (IViewEngine for one), in that case evaluate the best pattern (factory, type activation, service).

We want to keep extension methods that we've built where the do something useful. In general the properties on options will be an IList<T> and will be easier to work with overall.

@danroth27 danroth27 added this to the 6.0.0-beta5 milestone Mar 25, 2015
pranavkm added a commit that referenced this issue Apr 3, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 3, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 3, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 3, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 3, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 3, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 3, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 5, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 5, 2015
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
pranavkm added a commit that referenced this issue Apr 7, 2015
Follow up work item to #2269
pranavkm added a commit that referenced this issue Apr 7, 2015
Follow up work item to #2269
pranavkm added a commit that referenced this issue Apr 8, 2015
Follow up work item to #2269
pranavkm added a commit that referenced this issue Jul 29, 2015
pranavkm added a commit that referenced this issue Jul 29, 2015
pranavkm added a commit that referenced this issue Jul 29, 2015
pranavkm added a commit that referenced this issue Jul 29, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants