Azure-Samples / active-directory-dotnet-webapp-webapi-multitenant-openidconnect

A sample .NET 4.5 MVC SaaS web app that signs-up and signs-in users from any Azure AD tenant, and calls the Azure AD Graph API.
66 stars 45 forks source link

Feedback from Dan's intern #18

Closed BryanLa closed 5 years ago

BryanLa commented 9 years ago

Github Sample:

o Per instructions, I downloaded Visual Studio 2013. I assume there are other versions of Visual Studio that would have sufficed as well? I already had Visual Studio 2010, couldn’t I have just used that? o Step 3.9, as someone who had never made an AAD Tenant before, and had never used the portal before, I had no idea what my domain was at this step, and didn’t know where to go to find it. I was able to figure it out eventually because it I saw my domain in the page’s URL, but that was a weird way to figure it out o The bar at the bottom of the Azure Management Portal page is referred to in 3 different ways, which threw me off a bit: “the drawer”, “the command bar”, and “bottom command bar o Step 3.13, “Enable sign-on and read users’ profiles” was actually the default Permission, so I didn’t actually have to do this step. I think it’s still good to draw attention to what the set permissions are though, so I wouldn’t propose getting rid of this step altogether. o Step 4.4, what is a clipboard? Also, the directions say to “use” the key “now,” I think what was actually meant was that I needed to save the key now o Step 5, I did not know what “solution” meant, nor did I know how to open it in VS  In this step, managed to get pretty confused, somehow involving my opening the wrong web.config file, therefore incorrectly changing the ClientID and Password values, etc. Overall some more specific instructions in this step would’ve been useful o Step 6, “click Add”, there is no Add button, there’s a “New+” button though

kalyankrishna1 commented 5 years ago

The readme has been updated quite a few times since then. Not sure if these are valid concerns now. Closing for now.