Skip to content

Change anchor element to link to HelloWorld controller - #37597

Open
manishm004 wants to merge 1 commit into
dotnet:mainfrom
manishm004:patch-4
Open

Change anchor element to link to HelloWorld controller#37597
manishm004 wants to merge 1 commit into
dotnet:mainfrom
manishm004:patch-4

Conversation

@manishm004

@manishm004 manishm004 commented Sep 5, 2026

Copy link
Copy Markdown

Updated the anchor element in the markup to link to the HelloWorld controller instead of the Movies controller. Removed note about the Movies controller not being implemented.

Originally, the text below the code said:

The preceding markup made the following changes:
-     Three occurrences of MvcMovie to Movie App.
-     The anchor element `<a class="navbar-brand" asp-area="" asp-controller="Home" asp-action="Index">MvcMovie</a>` to `<a class="navbar-brand" asp-controller="Movies" asp-action="Index">Movie App</a>`.

But the line of code referred by the second point was:
<a class="navbar-brand" asp-area="" asp-controller="HelloWorld" asp-action="Index">Movie App</a>
and not:
<a class="navbar-brand" asp-controller="Movies" asp-action="Index">Movie App</a>

Hence the text below the code must say:

The preceding markup made the following changes:
-     Three occurrences of MvcMovie to Movie App.
-     The anchor element `<a class="navbar-brand" asp-area="" asp-controller="Home" asp-action="Index">MvcMovie</a>` to `<a class="navbar-brand" asp-area="" asp-controller="HelloWorld" asp-action="Index">Movie App</a>`.

And hence it is also necessary that the below line be removed from the documentation (Since there is no "Movie App" link , i.e. asp-controller="Movies", in the code as of now. Instead, the link in the code is of "HelloWorld", i.e. asp-controller="HelloWorld", which has already been implemented):

Note: The Movies controller hasn't been implemented. At this point, the Movie App link isn't functional.

Internal previews

File Preview link
aspnetcore/tutorials/first-mvc-app/adding-view/includes/adding-view9.md Learn preview

Build report

Updated the anchor element in the markup to link to the HelloWorld controller instead of the Movies controller. Removed note about the Movies controller not being implemented.
@wadepickett

wadepickett commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

@manishm004, This looks great. Thanks for making this fix!

There is one more file (version) to fix, if you like:
The same fix also needs to be extended to the .NET 10+ version here:
aspnetcore/tutorials/first-mvc-app/adding-view.md

I checked and the fix does not need to occur on any of the other previious versions.

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.

2 participants