AspNetCore.Docs icon indicating copy to clipboard operation
AspNetCore.Docs copied to clipboard

.NET 7: MapGroup support for more extension methods

Open Rick-Anderson opened this issue 3 years ago • 2 comments

MapGroup support for more extension methods

Related to #26476

Rick-Anderson avatar Jul 26 '22 00:07 Rick-Anderson

I think adding more extension methods to the Todo would be nice to have. Would make some updates

sammychinedu2ky avatar Jul 26 '22 21:07 sammychinedu2ky

@Rick-Anderson

sammychinedu2ky avatar Jul 26 '22 21:07 sammychinedu2ky

@sammychinedu2ky what's the status on MapGroup support for more extension methods Was this ever done?

Rick-Anderson avatar Sep 17 '22 01:09 Rick-Anderson

@sammychinedu2ky what's the status on MapGroup support for more extension methods Was this ever done?

https://github.com/dotnet/AspNetCore.Docs/tree/main/aspnetcore/fundamentals/minimal-apis/7.0-samples/todo-group

I wrote this a month ago.

sammychinedu2ky avatar Sep 17 '22 04:09 sammychinedu2ky

@Rick-Anderson I think it would be less confusing if I keep the todo-group sample as it is. For the unit/integration testing task, I've created another project WebMinRouteGroup, which is pretty much similar to @sammychinedu2ky sample (but updated)

@sammychinedu2ky you might want to update your todo-group sample since it's not working. Some of the APIs has changed e.g. AddRouteHandlerFilter has been renamed to AddEndpointFilter

once you update the sample, that could be used for showcasing the Route Group feature in the doc.

fiyazbinhasan avatar Sep 17 '22 07:09 fiyazbinhasan

Ooh thanks. Let me check it out @fiyazbinhasan

sammychinedu2ky avatar Sep 17 '22 07:09 sammychinedu2ky

done 🥂

sammychinedu2ky avatar Sep 17 '22 08:09 sammychinedu2ky

done 🥂

PR that updated the code?

Rick-Anderson avatar Sep 19 '22 21:09 Rick-Anderson

done 🥂

PR that updated the code?

Yh

sammychinedu2ky avatar Sep 20 '22 04:09 sammychinedu2ky