search

dotnet / docs-aspire

This repository contains .NET Aspire documentation.
https://learn.microsoft.com/dotnet/aspire
MIT License
87 stars 113 forks source link

[WebToolsE2E][Aspire][Document]Some more updates are needed to "Tutorial: Deploy a .NET Aspire app with a SQL Server Database to Azure" & "Deploy a .NET Aspire app to Azure Container Apps" instructions using CLI with aspire #1353

Open vmykagapuz opened 4 months ago

vmykagapuz commented 4 months ago

Describe the issue or suggestion

Issue description In "Tutorial: Deploy a .NET Aspire app with a SQL Server Database to Azure" & "Deploy a .NET Aspire app to Azure Container Apps" document, we need to make some updates to the instructions for 8.1 Preview 1.

Issue 1: For Create the sample solution section, the project name needs to be updated in the command to be match with the solution project's name.

Doc1

Issue 2: For Deploy the app section, the document shows "By default, a service can only be reached from inside the Azure Container Apps environment it is running in. Selecting a service here will also allow it to be reached from the Internet. ? Select which services to expose to the Internet webfrontend"

Doc3

Actual: The first 2 line is not shown after deploying the project and there is an additional aspire dashboard that needs to be updated in the document. Doc2

Associated WorkItem - 308346

sunnynagavo commented 3 months ago

@CamSoper

for issue 1, https://github.com/dotnet/docs-aspire/compare/main...sunnynagavo:docs-aspire:patch-1 for issue 2, last commit was in May 2024, may be the output response wording has changed since then and its pasted as a screenshot

CamSoper commented 3 months ago

Thank you, @sunnynagavo!

sunnynagavo commented 3 months ago

https://github.com/dotnet/docs-aspire/pull/1576