MicrosoftLearning / MicrosoftLearning/mslearn-devops
advanced /06-enable-dynamic-configuration-feature-flags.md (Lab 11 on cloudslice): casing issue
Nobody has claimed this yet.
- Dominant language
- PowerShell
- Stars
- 39
- Forks
- 59
- PR merge metrics
- No merged PRs in 30d
Description
Lab: https://github.com/MicrosoftLearning/mslearn-devops/blob/main/Instructions/advanced/06-enable-dynamic-configuration-feature-flags.md
Section: https://github.com/MicrosoftLearning/mslearn-devops/blob/main/Instructions/advanced/06-enable-dynamic-configuration-feature-flags.md#add-app-configuration-environment-variables
There is mention of:
Name: AppConfigEndPoint --> capital P
However the code is using AppConfigEndpoint --> lower case p
Because of this casing mismatch this environment variable isn't loaded correctly and there is an error in the lab (basically the feature flag seems to not work.)
Changing this casing to be the same both in the code and in the environment variable in app service fixes this error.
Please change the casing in the instructions of this lab.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open Instructions/advanced/06-enable-dynamic-configuration-feature-flags.md and review the “Add App Configuration Environment Variables” section. Make the AppConfigEndPoint/AppConfigEndpoint casing consistent with the lab’s code and environment-variable configuration, then verify the feature flag loads without the reported error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure
- Domain
- cloud, documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 91/100