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

Please Update Web Activator Dependency #29

Closed
LorenDorez opened this issue Jun 20, 2013 · 5 comments
Closed

Please Update Web Activator Dependency #29

LorenDorez opened this issue Jun 20, 2013 · 5 comments

Comments

@LorenDorez
Copy link

Hi I have been working with the creator of WebActivator closely for a new issue that arose with update to .net 4.5 using MVC. We have patched a work around in WebActivator 2.0.2, but ninject this requires 1.5.x

Can you please update ninject to use the latest WebActivator

@remogloor
Copy link
Member

Implemented in ninject/Ninject.Web.Common@55829c2

@johanncampbell
Copy link

Seems that there are still issues resolving WebActivator. I think it should be looking for WebActivatorEx (for Ninject.MVC3 3.0.2-unstable 8 on Nuget).

@lstyles
Copy link

lstyles commented Jan 6, 2014

Not only it should be using WebActivatorEx package instead. It should also use:

[assembly: WebActivatorEx.PreApplicationStartMethod(typeof(WebApplication1.Web.AppStart), "Start")]
[assembly: WebActivatorEx.ApplicationShutdownMethodAttribute(typeof(WebApplication1.Web.AppStart), "Stop")]

in NinjectWebCommon.cs that's automatically added.

Please update.

@danielmarbach
Copy link
Member

How about sending a pull?

Am 06.01.2014 um 04:10 schrieb lpaluszk [email protected]:

Not only it should be using WebActivatorEx package instead. It should also use:

[assembly: WebActivatorEx.PreApplicationStartMethod(typeof(VoiceVue.Web.AppStart), "Start")]
[assembly: WebActivatorEx.ApplicationShutdownMethodAttribute(typeof(VoiceVue.Web.AppStart), "Stop")]

in NinjectWebCommon.cs that's automatically added.

Please update.


Reply to this email directly or view it on GitHub.

@lstyles
Copy link

lstyles commented Jan 7, 2014

The old WebActivator reference comes from Ninject.Web.Common and not this repo. Apologies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants