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

MvcHostOptions need to be injected #774

Closed
pranavkm opened this issue Jul 10, 2014 · 2 comments
Closed

MvcHostOptions need to be injected #774

pranavkm opened this issue Jul 10, 2014 · 2 comments
Assignees
@pranavkm
Labels
3 - Done feature needs design
Milestone
1.0.0-beta1

Comments

@pranavkm
Copy link
Contributor

We need to support MvcHostOptions as a top level construct that is set up from Microsoft.AspNet.Mvc. This would allow users to set up default namespaces, base type etc
@pranavkm pranavkm mentioned this issue Jul 10, 2014
Closed
@pranavkm pranavkm mentioned this issue Jul 28, 2014
Closed
@pranavkm pranavkm self-assigned this Jul 28, 2014
@pranavkm pranavkm added this to the 6.0.0-rc1 milestone Jul 28, 2014
@pranavkm pranavkm modified the milestones: 6.0.0-rc1, 6.0.0-beta1 Jul 29, 2014
@pranavkm
Copy link
Contributor Author

One of the issues we have with DI in the host is that we need to continue providing design time support even if the application does not compile.
Discussed this with @yishaigalatzer and we settled on attempting to read these values from the DI if available or falling back to using sensible defaults when we can't.

pranavkm added a commit that referenced this issue Jul 29, 2014
@pranavkm
Consume MvcRazorHostOptions from the service container if available
a0efdd8 
Fixes #774
@pranavkm pranavkm mentioned this issue Jul 29, 2014
Closed
@danroth27
Copy link
Member

We are going to move everything to a RazorHostFactory. The factory is going to have no dependencies on the user's code startup.cs.

@tactical-drone tactical-drone mentioned this issue Jun 7, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pranavkm pranavkm

Labels
3 - Done feature needs design
Projects
None yet
Milestone
1.0.0-beta1
Development

No branches or pull requests
2 participants
@pranavkm @danroth27