Create Azure App Service
 “Login to Azure portal” – how / where?
 Please describe how to get a free trial subscription.

"Source Code: You can find the initial source code here (TODO link with final URL)."

"title: Drive user engagement across all your devices with Microsoft Graph"

• Should each word be capitalized?

"In this lab your learn what is graph, how use it and the possibilities that offers us. You write C# code and see tthe results in UWP App."

• Phrasing is a bit off here as well.

"title: Integrating MS Graph into you app"

Unable to get sample project from following link:

Download this assets.zip file and unpack the images and JSON to the lunchmenu\src\assets folder.

On the landing page, the "Start Lab" links for machine learning and graph point to the wrong labs.

"In these labs you will be introduced how to build a Progressive web application".

would be better as:

"This lab will explain how to build a Progressive Web App (PWA)".

https://review.docs.microsoft.com/en-us/labs/insiderdevtour2018/graph/index?branch=insiderdevtour%2Fdev&ocid=idt_labs_cta_web2lab_graph&tutorial-step=2

In the Get call to Get all items in OneDrive section, I'm not seeing any onedrive items when I run the sample:

Graph Explorer earlier showed me items with this call:
https://graph.microsoft.com/v1.0/$metadata#users('<userid>')/drive/root/children

So, I'm not sure what's wrong, the request doesn't contain any items:

Also, should these calls be using the async method pattern rather than .Result?

Example:

Emotion detection with Face API (Lab-part-2.md) should become something like:

Emotion detection with Face API (Cloud) (Lab-part-2.md)
Emotion detection with Windows ML (edge) (Lab-part-2b.md)

And the .yml file will need to be updated accordingly.

Distribution and versioning should come first as some Windows 10 APIs require a package identity.

1. Distribution and versioning
2. UI modernization
3. Windows 10 APIs - Toast
4. Adaptive cards

https://review.docs.microsoft.com/en-us/labs/insiderdevtour2018/graph/index?branch=insiderdevtour%2Fdev&ocid=idt_labs_cta_web2lab_graph&tutorial-step=2

Under 'Sample API calls' and Create a console app it doesn't link to the project directory like the previous sample did. It should link here I'm assuming: https://github.com/Microsoft/InsiderDevTour18-Labs/tree/master/graph/Microsoft.Graph.HOL.ConsoleBase

It could be useful to put the git clone link in the documentation as well?

(Also, should this section be broken up a bit. We do the app setup, then we do this separate investigation in the graph explorer, and then we go to the console app...?)

For the "Emotion Detection with Face API" section, I can't find Microsoft.Toolkit.Uwp.UI.Controls (v3.0.0) in VS, so CameraPreview doesn't exist in my project.

Adjust .md/.yml files to give the following:

"Exercise 1: Create a Team" (it's own .md file - and remove the "Exercise 1" numbering)
"Graph APIs" (it's own .md file - and remove the "Exercise 2" numbering)
"Canvases across Office 365" (it's own .md file)

There's a lot of great stuff called out in the Why is this useful section of the Graph introduction, but it'd be great to actually link to samples/docs showing how to do these various things. Otherwise it's just taunting or frustrating if those are things I know I want to do.

On Edge 17134.

When you're at the bottom of a page and hit the 'Next' button:

You're brought to the next page, but at the bottom:

I'd expect to be brought to the top of the next article instead:

This happens in all labs.

@cbarkerms is this something we have control over or is part of the docs code?

We need to show an example of calling into Windows 10 APIs from a Desktop app - example: Toast notifications

"Getting Started with Quick Start" onwards should be on it's own page. Please also update the .yml to reflect the update in structure.

The term "HMDU MR experience" is used in the Introduction. HMDU is not a recognized term. Even if it was meant to be HMD (for Head Mounted Display), that is not recommended. "Mixed Reality Headset" is the preferred term.

BTW, do not google HMDU. This is a Pri 0 Policheck issue.

The instructions state:

To deploy the project to an immersive headset.
Navigate to the Build Window: Mixed Reality Toolkit -> Build Window.

Click on Open in Visual Studio.

Note: If Unity says there is no "open project solution" and asks if you still want to build, select "Yes, 
Build"

Navigate to Build -> Configuration Manager in Visual Studios or use the drop-down at the top of the 
page and select x64 in your platform options.

In the debug toolbar, select "Local Machine" as the target (or remote device if you are pushing to 
another machine).

Run the app by hitting play.

I ran into the following issues:

Click on Open in Visual Studio.

Visual Studio did not open for me. I had to click on Open in Visual Studio again after the export of the project completed.

Run the app by hitting play.

If you hit play you will get build errors with several missing methods. Selecting Build Solution from the Build menu resulted in a successful build

If its a permission issue, note that this has to be enabled when using Graph API

• Note *
  I am working through the content, and will submit a PR with some grammar improvements.

This lab really needs a good copy edit. Here are some suggestions for the PWA Foundations page.

"Technologically speaking" - remove this text.

"Standard Manifest: The site should be controlled by a W3C manifest that determines the experience and behavior of your PWA"

would be better as

"Standard manifest file: The site requires a manifest file to define the features and behavior of the PWA. The file follows the W3C standard".

"Since at their core PWAs are still websites, they’re already supported on numerous platforms like Windows, Android, iOS, and Mac. They can be enhanced over time to function more like native apps, and even use native APIs, on supported platforms. This can turn your website into a Rockstar app by combining the best of the web and native application worlds. Plus, the work of turning your website into a PWA can also improve your website’s experience."

would be better as:

"PWA apps are still websites, and are therefore supported on most platforms including Windows, Android, iOS and Mac. They can enhanced to function more like native apps, and even use native APIs on support platforms. A PWA can combine the best features of a web site and a native application."

i.e. removing "Rockstar app"

"PWAs aren't very different that a web application, when you access to a PWA you will see the same as a simple web application. After complete this lab, your PWA will looks like this:"

ok, I give up :-) This sentence alone has about 10 issues :-)

In 4. Environment Setup (VR)

"Next, we need to choice an experience scale" - choice should be choose

"Select the MixedRealityCameraParent in the hierarchy and change the Transform -> Position -> Y to 0.8. This will move the camera to a natural eye level." - The image shows 0.0 for the Transform -> Position -> Y. It should show 0.8.

If this was to allow a demo account we could use for testing, it's now broken.
"Sign-in is blocked. You've tried to sign in too many times with an incorrect account or password"

There are broken pictures/missing images scattered throughout the different pages. Right now there are small black boxes with "x"s in their places.

Please see the documentation shared previously for details.

“In Teams, card can be shown in chats with bots” – should be “cards”.

At the end of the “Exercise 1: Create a Team” page, you have a section on “Graph APIs” – It doesn’t seem to flow naturally from the previous point (number 11). Can you please be a bit more descriptive about what you will do next – example: “read on to see what a developer can do with the Microsoft Graph”.

Publish your PWA

The instructions say "Pull the "manifest.json" file and the "images" folder that are inside of the PWA folder "
It is unclear what "pull" means in this context. Delete? Copy?

PWA on Windows
Broken images.

Some later pages in the git repo don't appear to be published at all, at least they aren't in the TOC. Maybe by design, as they look a little rough.

Was on the last step of https://review.docs.microsoft.com/en-us/labs/insiderdevtour2018/graph/index?tutorial-step=2&branch=insiderdevtour%2Fdev

I'm getting this can't connect to service after typing in my username and 'getting redirected to organization sign-in page' (the console app is still working fine, as does logging in a browser).

If I close the window, the page contents has a Microsoft.Identity.Client.MsalException: WAB authentication failed followed by a FileNotFoundException.

For the following:

"Click on the Test and Distribute selector from the left panel under Distribute and then the Export button to download and save the LunchMenu.zip manifest file to your desktop."

The download location was the "Downloads" folder.

@fpelaez - I already did this for lab-part-0 -> lab-part-2. Just lab-part-3 is left so please complete that and use 0-2 as a reference. thanks!

The docs show this for the default settings:

My fresh install of Unity 2017.4.2 and fresh import of the Holotoolkit.unitypackage displayed this:

Note: Use Toolkit-specific InputManager axes was NOT enabled by default in my fresh install.

It would be better to state the exact settings you require to be selected.

Also user should click Apply

I am getting different options from the images in the docs. The docs show this for the "Apply Mixed Reality Scene Settings"

The Holotoolkit.unitypackage from the InsiderDevTour18-Labs shows this:

Did you use the same version of the Holotoolkit.unitypackage when you created the docs?

This step comprises of 3 images - please add text explaining the steps in each - images only would be an issue for people with poor eyesight.

Also - please explain how to obtain a GUID.

1. Helpers/OutlookHelper.cs SetAppintment should be SetAppointment. Correct spelling in code and the lab as well.

2. In the app, both Date Time say Start. One should say Date Time Stop. Also update image in lab.

3. After scheduling event in app, it creates the event on the outlook calender but the subject is incorrect. It doesn't show the subject provided in the app, it always creates an event with Subject "Subject"

In this section-
Choosing the project template
Now we will download the UWP application. We will use it and configure it with the Application Id and the Redirect Uri that we saved before.

It asks to download project from here. But there are two projects under graph folder. Doesn't indicate which project to open.

Under “Exercise 1: Create a Team - In the Cafeteria channel, click on Files and Upload a file.”, it is not clear if I should be using a particular file or file type.

When following these instructions:

To deploy the project to an immersive headset.
Navigate to the Build Window: Mixed Reality Toolkit -> Build Window.
Click on Open in Visual Studio.
Navigate to Build -> Configuration Manager in Visual Studios or use the drop-down at the top of the page and select x64 in your platform options.
In the debug toolbar, select "Local Machine" as the target (or remote device if you are pushing to another machine).
Run the app by hitting play.

I get the following error:

Need instructions on how to make sure the scene is added to the build.

Repro:

1. Start InPrivate session
2. From labs page
3. Click Graph's Start Lab
4. Login
5. Let doc page load
6. Hit back button in browser.

Expected

Return to lab selection page

Actual

Get 403 forbidden page as in image above.

Not sure if this matters since there's permissions on it currently, but figured I'd report it, just in case.

Examples:

MS Graph => either "Graph" or "Microsoft Graph"
"In UWP project go to Helpers/OneDriverHelper.cs" => "In the UWP project go to …"
"Integrating with office" => "Integrating with Office"
"Advance app feature" => "Cortana integration"

The images are not rendering.

I think it'd be nicer if each task in the integration section was broken up on it's own page. This way you could move the Timeline paragraph from the Introduction page as the intro to the Timeline activity. I think it'd make more sense there in context with performing the activity right after. Otherwise, you read about timeline in the intro and will forget about it by the time you get to the activity.

• Using OneDrive for app data storage
• Writing data to the MS Graph
• Integrating with office
• Writing to the Timeline (I'd probably put the deeplink in here and then end with the pick-up where you left off as all part of this same module, as they're all interconnected)

In the "Exporting a WinMR UWP app from Unity" section:

• "WinMR" and "Win MR" are not the correct terms. Use Windows Mixed Reality.

• The first step requires importing files from the "lab files". There is no link to these files, and so I am
  blocked from getting any further.

• Use of the phrase "super simple" is discouraged. It may be "super simple" to the author, but not the person trying to do it.

On the PWA foundations page, it shows some example comparisons of websites and their app conversion.

However, they all just show the landing page of the website, which isn't very interesting. It'd be better to show an actual Twitter feed, a Trivago search, or a DayTrip activity.

First, got red squiggles when opening project (they went away after I built, but may want to call it out).

More importantly, I followed the steps and got this:

App just hung after I closed the window and didn't do anything, what do I do?

I have my app id and return url in <appSettings> in the app.config under the root <configuration> element.