JSkimming / AspNet.Identity.EntityFramework.Multitenant

Multi-tenant support for ASP.NET Identity using Entity Framework
Apache License 2.0
132 stars 61 forks source link

tenantId part of UserNameIndex is defined in concrete MultitenantIdentityDbContext instead of in generic MultitenantIdentityDbContext #7

Closed lukasz-karolewski closed 9 years ago

lukasz-karolewski commented 9 years ago

Hi i was wondering why did you split UserNameIndex definition into 2 files. I recently moved over to inheriting from generic MultitenantIdentityDbContext and i was really unexpected that suddenly UserNameIndex contains only index on UserName instead of (tenantId, UserName)

have you have any particular reason to split it up like that?

Thanks, Lucas

lukasz-karolewski commented 9 years ago

i guess i should call it a day today ;) sorry for spamming, obviously you cant know the type... of TTenantKey in generic version

JSkimming commented 9 years ago

:smiley: @kether667 the confusion could be addressed with better documentation. I'd be happy to accept PRs with help in that area.