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

Remove facades for accessing Options<T> and pass options to the invoker #2266

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

Remove facades for accessing Options<T> and pass options to the invoker #2266

rynowak opened this issue Mar 25, 2015 · 0 comments
Assignees
@pranavkm
Labels
3 - Done API review enhancement
Milestone
1.0.0-beta5

Comments

@rynowak
Copy link
Member

rynowak commented Mar 25, 2015

From API review:

Reduce the numbers of services declared by passing options directly to the invoker. There's a bunch of facades (IModelBinderProvider, IOutputFormatterProvider, etc...) which abstract over options and really just provide access to a collection.
@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
@pranavkm
* Simplify MvcOptions
4613d37 
* 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
@pranavkm
* Simplify MvcOptions
8f03f43 
* 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
@pranavkm
* Simplify MvcOptions
1b8fa1c 
* 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
@pranavkm
* Simplify MvcOptions
5fb40e8 
* 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
@pranavkm
* Simplify MvcOptions
8c0c592 
* 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
@pranavkm
* Simplify MvcOptions
85e006b 
* 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
@pranavkm
* Simplify MvcOptions
2092aea 
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
@pranavkm pranavkm mentioned this issue Apr 3, 2015
Closed
pranavkm added a commit that referenced this issue Apr 5, 2015
@pranavkm
* Simplify MvcOptions
29ac06d 
* 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
@pranavkm
* Simplify MvcOptions
4374a06 
* Remove facades for accessing Options<T> and pass options to the invoker

Fixes #2266
Fixes #2269
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pranavkm pranavkm

Labels
3 - Done API review enhancement
Projects
None yet
Milestone
1.0.0-beta5
Development

No branches or pull requests
3 participants
@pranavkm @rynowak @danroth27