Chris Martinez

Results 311 comments of Chris Martinez
trafficstars

@akshaybheda, that looks like the OData route debugging page. When you version by URL segment, API Versioning does not know where API version will appear. It _could_ be `v{ver:apiVersion}`, `api/v{ver:apiVersion}`,...

Expanding as I've discovered the same issue, albeit a different way. This problem also arises if you use the xUnit [methodDisplayOptions](https://xunit.net/docs/configuration-files#methodDisplayOptions) feature. @stefanforsberg I agree that the tooling seems pretty...

Hmm... I'm not able to repro your condition. I verified from the current examples and it worked. Then I took your repro verbatim and it also works. I tried the...

Hmmm... the explanation threw me off a bit, but I see the issue now. It seems like it is related specifically to the combination of `HeaderApiVersionReader` and `AssumeDefaultVersionWhenUnspecified`. ![image](https://github.com/dotnet/aspnet-api-versioning/assets/11765008/4ecd9565-04d9-445a-a172-a8a80719c61c) I'm...

This is intentional and by design. The new approach was outlined in the [roadmap 2 years ago](../discussions/808). The full gambit is: ```c# services.AddApiVersioning() // Core API Versioning services with support...

In the wiki, there is already the text: ``` .AddMvc(); // ← brings in MVC Core; unnecessary for Minimal APIs ``` There's even an entire page dedicated to [migration](../wiki/Migration) that...

Yep, I get the surprise and there was no great way to fully advertise that. _Behavioral_ changes always concern me the most because they are the least obvious and hardest...

[POLA](https://en.wikipedia.org/wiki/Principle_of_least_astonishment) is part of every decision. Unfortunately, it's just not always easy to advertise changes.

On the line: https://github.com/GProGamer/HotelListing/blob/master/HotelListing.API/Program.cs#L52 you need to add `.AddMvc()`. Starting in API Versioning `6.0`, this call is now required to bring in MVC Core. Without this call, only _Minimal APIs_...

I browsed over the repro a little _too_ fast. There's actually a number of things missing from your configuration that _should_ be there. This information is provided in the wiki...