httpflow icon indicating copy to clipboard operation
httpflow copied to clipboard

Add Identity/Auth to template #300

Open ahmedanwar100 opened this issue 2 years ago • 9 comments

Jwt Authentication added with default Microsoft Identity

ahmedanwar100 avatar Jan 01 '23 14:01 ahmedanwar100

Hi @ardalis, can you please review it

ahmedanwar100 avatar Jan 01 '23 14:01 ahmedanwar100

Can you give me time till sunday?

On Thu, 13 Apr 2023, 7:14 pm Steve Smith, @.***> wrote:

@.**** requested changes on this pull request.

Overall looks great, thanks!

Only request is to don't put Identity stuff in Core (instead you can reference it using a UserId property on an appropriate entity, like in our case perhaps Contributor) and also keep IdentityDbContext separate from the regular/existing AppdbContext. Can you make those changes and I'll merge?

— Reply to this email directly, view it on GitHub https://github.com/ardalis/CleanArchitecture/pull/487#pullrequestreview-1383521649, or unsubscribe https://github.com/notifications/unsubscribe-auth/AILYWTSUSE4B4XJVZ7VJTBTXBAC5DANCNFSM6AAAAAATOEIRPM . You are receiving this because you authored the thread.Message ID: @.***>

ahmedanwar100 avatar Apr 13 '23 14:04 ahmedanwar100

I will try to complete it before sunday

On Thu, 13 Apr 2023, 7:31 pm Ahmed Anwer, @.***> wrote:

Can you give me time till sunday?

On Thu, 13 Apr 2023, 7:14 pm Steve Smith, @.***> wrote:

@.**** requested changes on this pull request.

Overall looks great, thanks!

Only request is to don't put Identity stuff in Core (instead you can reference it using a UserId property on an appropriate entity, like in our case perhaps Contributor) and also keep IdentityDbContext separate from the regular/existing AppdbContext. Can you make those changes and I'll merge?

— Reply to this email directly, view it on GitHub https://github.com/ardalis/CleanArchitecture/pull/487#pullrequestreview-1383521649, or unsubscribe https://github.com/notifications/unsubscribe-auth/AILYWTSUSE4B4XJVZ7VJTBTXBAC5DANCNFSM6AAAAAATOEIRPM . You are receiving this because you authored the thread.Message ID: @.***>

ahmedanwar100 avatar Apr 13 '23 14:04 ahmedanwar100

No rush, thanks

ardalis avatar Apr 13 '23 14:04 ardalis

Great, thanks!

On Thu, 13 Apr 2023, 7:48 pm Steve Smith, @.***> wrote:

No rush, thanks

— Reply to this email directly, view it on GitHub https://github.com/ardalis/CleanArchitecture/pull/487#issuecomment-1507105133, or unsubscribe https://github.com/notifications/unsubscribe-auth/AILYWTXEQ3IDQCFZWGXRBF3XBAG3LANCNFSM6AAAAAATOEIRPM . You are receiving this because you authored the thread.Message ID: @.***>

ahmedanwar100 avatar Apr 13 '23 14:04 ahmedanwar100

@ardalis can you please review it

ahmedanwar100 avatar Apr 22 '23 03:04 ahmedanwar100

@ardalis, any plan to review it 😁

ahmedanwar100 avatar Jul 23 '23 20:07 ahmedanwar100

My plan is still to pull this into the new template - just need to find some time.

ardalis avatar Sep 01 '23 16:09 ardalis

Okay, then please find some time 🙂

ahmedanwar100 avatar Sep 01 '23 16:09 ahmedanwar100