giorgos07 / Daarto

Dapper implementation of ASP.NET Core Identity stores.
260 stars 51 forks source link

String should be TKey in DapperStoreOptionsExtentions to use different ID type #15

Open SZLMCL opened 4 years ago

SZLMCL commented 4 years ago

Hello,

I would like to use Guid ID type as ID of Users and Roles...

In the current extension method the 'string' type is hardcoded in more place (one example):

    public static void AddUserRolesTable<TUserRolesTable, TUserRole>(this DapperStoreOptions options)
        where TUserRolesTable : UserRolesTable<IdentityRole, string, TUserRole>
        where TUserRole : IdentityUserRole<string>, new() {
        options.AddUserRolesTable<TUserRolesTable, IdentityRole, string, TUserRole>();

I think the string type should be replaced with TKey.