Add external authentication (MSA) and users #82
Conversation
asyasky
force-pushed
the
localAuthAndRoles
branch
from
2ef7329
to
366765b
Compare
asyasky
force-pushed
the
localAuthAndRoles
branch
from
366765b
to
3168e67
Compare
| <hr /> | ||
| <div class="row"> | ||
| <div class="col-md-3"> | ||
| <partial name="_ManageNav" /> |
There was a problem hiding this comment.
We may want to make our menu into a partial like this (first I've heard of this feature, I assume this is an instruction to include _ManageNav.cshtml) and include it in this page.
There was a problem hiding this comment.
That seems like a good plan for standardizing across pages.
There was a problem hiding this comment.
I think that's what _Layout.cshtml is as well
| public string EmployeeAlias { get; set; } | ||
| public string Name { get; set; } | ||
|
|
||
| [Display(Name = "T-shirt size")] |
There was a problem hiding this comment.
Should this be conditional on intern events or other event properties?
There was a problem hiding this comment.
Maybe, maybe not - the user persists between events, it's not linked to events. The current site has this visible all the time.
| @@ -0,0 +1,26 @@ | |||
| Support for ASP.NET Core Identity was added to your project | |||
There was a problem hiding this comment.
I'd like to leave it in because there are things mentioned that don't seem to be documented other places (e.g. that order matters).
asyasky
force-pushed
the
localAuthAndRoles
branch
from
c6f10be
to
b3f04ba
Compare
Allows profile creation and login using Microsoft Accounts.
#23