Skip to content

Conversation

@sander1095
Copy link
Contributor

@sander1095 sander1095 commented Nov 11, 2025

@sander1095
Copy link
Contributor Author

Note: Should this PR also include removing this comment:

https://github.com/dotnet/AspNetCore.Docs/blob/a7b09b2cca9f23d241350ac0b3d8917aa3b6904e/aspnetcore/fundamentals/openapi/customize-openapi.md?plain=1#L105C1-L111C114

I can also make this a separate PR :)

@wadepickett

@wadepickett
Copy link
Contributor

Note: Should this PR also include removing this comment:

https://github.com/dotnet/AspNetCore.Docs/blob/a7b09b2cca9f23d241350ac0b3d8917aa3b6904e/aspnetcore/fundamentals/openapi/customize-openapi.md?plain=1#L105C1-L111C114

I can also make this a separate PR :)

@wadepickett

Sure it can be the same PR, thanks!

@wadepickett
Copy link
Contributor

@sander1095,
We already have details on this topic, which could just be summarized in the What's new and point to the link below. If we needed to build up any more examples it would be added to this topic I think :

Customize OpenAPI output with operation transformers

Thoughts?

@sander1095
Copy link
Contributor Author

I don't really agree; I think the release note has a bit more body with a bit of code to show the example. However, I also agree that the information is already quite clear and present in the OpenAPI docs themselves.

To me, the important thing is that this is mentioned, so I'll implement your comment :)!

@sander1095
Copy link
Contributor Author

Done! I'll create a new issue to remove that commented xref, when I do that locally it doesn't look right. To prevent that blocking this release note, a new issue makes more sense: #36331

Copy link
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks @sander1095 for taking this on!

@wadepickett wadepickett merged commit 2045836 into dotnet:main Nov 12, 2025
4 checks passed
@wadepickett
Copy link
Contributor

@sander1095, please accept my apology. Were you the developer on this feature? I saw you were involved in .NET 10.

I didn't get a chance away from other work to respond before you changed the commit to remove the examples. If you want the What's New for .NET 10 doc to have a little more substance in the What's New announcment for this, that is perfectly fine. I just thought you didn't realize it was already covered in a topic. I can put the details you provided back in a What's New PR if you like. Up to you. Either was fine. We usually try to keep the What's New in summary form, and example is cool, but then the main content goes in the dedicated topic that is kept up to date with each relase.

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.

missing release notes about endpoint specific operation transformers in .net 10

2 participants