dddeastanglia / DDDEastAnglia

DDD East Anglia website
https://www.dddeastanglia.com
7 stars 10 forks source link

Remove a hangover from before we had Ninject wired in #263

Closed adrianbanks closed 10 years ago

adrianbanks commented 10 years ago

I'm a bit confused by this. I can only think that I don't understand how Ninject's convention-based binding works.

I appear to need a binding for:

IMailHostSettingsProvider -> SmtpHostSettingsProvider

but not for:

IMessageFactory - > SendGridMessageFactory IEmailSender -> SendGridEmailSender IAppSettingsProvider -> WebConfigurationAppSettingsProvider

I don't understand why the difference.

alastairs commented 10 years ago

I've had a look and can't figure it out as yet either :unamused:

adrianbanks commented 10 years ago

This change does work, but I don't really understand the difference in the configuration.