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

Update app.js #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Neutrino-Sunset
Copy link

Purpose

Fixes a bug that caused the website to fail to load correctly when hosted in a Linux container. This was caused by the path to the TodoList.html file using the wrong case.

Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type

[ x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
// Clone to a WSL directory
git clone https://github.com/Azure-Samples/dotnet-core-api
cd dotnet-core-api
git branch -m main
  • Test the code
Open VsCode.
Select 'Open Folder in Container'
Select options, C# runtime, version 3, no Azure CLI.
Once container is open in integrated terminal execute 'dotnet run'
Navigate to http:localhost:5000

What to Check

In Chrome Dev Tools Network tab verify that TodoList.html is no longer 404
View should now load corectly.

Other Information

Fixed a bug that caused the website to fail to load correctly when hosted in a Linux container. This was caused by the path to the TodoList.html file using the wrong case.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant