Skip to content

Conversation

@bart-vmware
Copy link
Contributor

This was already fixed in ASP.NET in dotnet/aspnetcore#59088.

Copilot AI review requested due to automatic review settings October 9, 2025 13:37
@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2025

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 11926

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 11926"

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a spelling error in project templates, correcting "Farenheit" to "Fahrenheit" in the aria-label attribute of weather display tables. This aligns with a similar fix that was already made in ASP.NET Core.

  • Corrects the misspelling of "Fahrenheit" in aria-label attributes
  • Updates both 9.5 and 13.0 versions of the Aspire starter template
  • Improves accessibility by providing accurate screen reader labels

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/Aspire.ProjectTemplates/templates/aspire-starter/9.5/Aspire-StarterApplication.1.Web/Components/Pages/Weather.razor Fixed spelling of "Fahrenheit" in aria-label for temperature column
src/Aspire.ProjectTemplates/templates/aspire-starter/13.0/Aspire-StarterApplication.1.Web/Components/Pages/Weather.razor Fixed spelling of "Fahrenheit" in aria-label for temperature column
Comments suppressed due to low confidence (1)

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Oct 9, 2025
@davidfowl davidfowl merged commit bf5f738 into dotnet:main Oct 9, 2025
303 checks passed
@davidfowl
Copy link
Member

Thanks!

@dotnet-policy-service dotnet-policy-service bot added this to the 13.0 milestone Oct 9, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Nov 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-templates community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants