Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating the hands-on labs to 2019/07 environments #9

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion continuous-deployment/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@ In this lab, you have an example MVC application, committed to a Git repository

### Pre-requisites: ###
- Complete [Continuous Integration](../continuous-integration/README.md) lab.

- Having your subscription linked with your own Azure Active Directory instance<br>
Check the [Course Pre-Requisites #1](../README.md#course-pre-requisites)

### Tasks Overview: ###

Expand Down
3 changes: 2 additions & 1 deletion feature-flag/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ Based on [this](https://microsoft.github.io/PartsUnlimited/apm/200.6x-APM-Featur
public void ConfigureServices(IServiceCollection services)
{
// Set provider config so file is read from content root path
var provider = new AppSettingsProvider { Configuration = (IConfiguration)Configuration };
var provider = new AppSettingsProvider { Configuration = (IConfigurationRoot)Configuration };

// Add your feature here
services.AddSingleton(new CheckPhoneNumber { ToggleValueProvider = provider });
Expand Down Expand Up @@ -164,6 +164,7 @@ Based on [this](https://microsoft.github.io/PartsUnlimited/apm/200.6x-APM-Featur
1. Enter phone number 123-123-5678, submit and notice the page refreshes without error

1. Push your code changes and let your pipeline do it's job
- Note (optional): You can override the feature toggle setting value defined in the appsettings.json file by using the Azure Web App settings screen (Azure portal > your Web app > Settings > Configuration panel). Open your production Web app configuration panel and try adding a new Application Setting called FeatureToggle:CheckPhoneNumber then assign it with either true or false. Hit the Save button and refresh the Contact page in your Production environment to check the feature toggle in action.

## Next steps

Expand Down
4 changes: 3 additions & 1 deletion getting-started/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@ In this lab, we will be installing the required development components and verif
[Azure Portal](https://portal.azure.com)
2. An active Azure DevOps account.<br>
[Sign up for Visual Studio Team Services](https://azure.microsoft.com/en-us/services/devops/)

3. Having your subscription linked with your own Azure Active Directory instance<br>
Check the [Course Pre-Requisites #1](../README.md#course-pre-requisites)

## Set up your machine ##
1. Install [Visual Studio 2017](http://go.microsoft.com/fwlink/?LinkId=517106)<br>
Select ASP.NET and web development and Azure development tools on the installer.
Expand Down
5 changes: 2 additions & 3 deletions source/deploy/azuredeploy.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
"2"
],
"defaultValue": "0"
},
}
},
"resources": [
{
Expand Down Expand Up @@ -62,8 +62,7 @@
"webSocketsEnabled": true,
"alwaysOn": true,
"remoteDebuggingEnabled": true,
"remoteDebuggingVersion": "VS2013",

"remoteDebuggingVersion": "VS2017",
"defaultDocuments": [
"index.html",
"hostingstart.html"
Expand Down