search

ThreeMammals / Ocelot

.NET API Gateway
https://www.nuget.org/packages/Ocelot
MIT License
8.24k stars 1.62k forks source link

Update "Getting Started" docs #2082

Closed GeeSuth closed 1 month ago

GeeSuth commented 1 month ago

As new to Ocelot I found that convinced to understand when you mention minimal API and code seems not.

Docs

raman-m commented 1 month ago

Hello, Abdallh! I'd accept this PR to update docs if you update Basic sample too. The Basic sample must work for .NET 6, 7 and 8!!!

raman-m commented 1 month ago

As new to Ocelot I found that convinced to understand when you mention minimal API and code seems not.

The documentation could be divided into two subsections: minimal API and non-minimal. If a minimal API design is not feasible, I would suggest omitting the phrase "minimal API" entirely from the documentation.

raman-m commented 1 month ago

You could return to us with more refined code in the feature branch.