Skip to content

Update Swashbuckle link #562

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

Merged
merged 1 commit into from
Jul 21, 2025
Merged

Update Swashbuckle link #562

merged 1 commit into from
Jul 21, 2025

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Jul 21, 2025

Fixes #561

Would you be able to make it redirect to the net 9 docs instead?

Thanks @DevTKSS! 🚀

cc: @BethMassi ... Recommend that you change the underlying link for https://aka.ms/aspnetcore/swashbuckle to not point at the 8.0 version of the article. At least for now, my fix is merely to place the absolute URL of the doc here without an article version.

@guardrex guardrex merged commit 5211afe into main Jul 21, 2025
2 checks passed
@guardrex guardrex deleted the guardrex-patch-1 branch July 21, 2025 10:46
@BethMassi
Copy link

@guardrex the destination link doesn't specify a version: https://learn.microsoft.com/en-us/aspnet/core/tutorials/getting-started-with-swashbuckle

@guardrex
Copy link
Collaborator Author

@BethMassi Ah! I see. Swashbuckle isn't available in .NET 9 or later. I only looked at the version selector and noticed it at "8.0" when I loaded the article.

I'll revert this! Sorry for my confusion on this one! 🙈😆

guardrex added a commit that referenced this pull request Jul 24, 2025
guardrex added a commit that referenced this pull request Jul 24, 2025
@DevTKSS
Copy link

DevTKSS commented Jul 24, 2025

@guardrex looked to me reading the docs md files, as if this problem might be caused by a xref link on that top Box 🤔
you do have a Include in this file here:

and this is what the "Asp Net Core Web API" net 8 asp net core template is providing to me, as-is from MS
image

So in my opinion, as this is proposing that we would get Swagger != Swashbuckle (?) which is up from net9 still this code there:
image

and this is also still the same provided (except from the Version of course):
image

Maybe that name of the aka link, the postfix there, should be updated to "swagger"? even in the currently as STS provided net8 templates?

I mean... The comment is essentially advertising, that we would use Swagger/OpenAPI but the argument now is, while we still use that same, thats something different? well at least thats strange 🤔 just my opinion.

but its your decision 👍

@guardrex
Copy link
Collaborator Author

They converted over to OpenAPI for the web API project template ...

https://github.com/dotnet/aspnetcore/blob/main/src/ProjectTemplates/Web.ProjectTemplates/content/WebApi-CSharp/Program.cs#L49-L50

... and we'll do the same here. I've opened #564 to work on it.

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.

Change Swashbuckle link
3 participants