Bart Koelman

A different (simpler) approach is described at https://radu-matei.com/blog/dark-mode/.

Sure, sounds good. The endpoints are defined at https://jsonapi.org/format/#fetching. Comments on controllers and services are in place already. See: https://github.com/json-api-dotnet/JsonApiDotNetCore/blob/952518a8c43c400445c790ebe285bf453949e71f/src/JsonApiDotNetCore/Controllers/BaseJsonApiController.cs#L126-L134 https://github.com/json-api-dotnet/JsonApiDotNetCore/blob/952518a8c43c400445c790ebe285bf453949e71f/src/JsonApiDotNetCore/Services/IGetSecondaryService.cs#L11-L15.

@verdie-g Would you like to create a PR for this?

Related: should set [OutputPath](https://stevetalkscode.co.uk/openapireference-commands#outputpath) for NSwag in the example project and/or LegacyOpenApiIntegration. This makes it easy to spot how the generated client changes when trying out different switches or updating...

Should include a test that verifies `public IDictionary AdditionalData { get; set; }` is _only_ generated for `meta` properties.

Should clean up references to Swashbuckle* packages.

I'd love to see this fixed as well. There's an [open PR](https://github.com/DapperLib/Dapper/pull/1608) that adds an overload that takes a `CommandDefinition`. This seems to be sufficient (and non-breaking) because a `CommandDefinition`...

Thanks for fixing this. I can confirm that #3043 is working properly now when using EF Core 9 preview 3 with the MyGet feed from https://www.myget.org/F/npgsql-vnext/api/v3/index.json (Npgsql.DependencyInjection v9.0.0-preview.1-ci.20240519T065219 and Npgsql.EntityFrameworkCore.PostgreSQL...

Ok sure, no problem. We'll wait until EF Core 9 then, before using this.

Possibly related: I found that replacing: ```c# string? connectionString = builder.Configuration.GetConnectionString("Default"); builder.Services.AddDbContext(options => options.UseNpgsql(connectionString)); ``` with: ```c# string? connectionString = builder.Configuration.GetConnectionString("Default"); builder.Services.AddNpgsqlDataSource(connectionString); builder.Services.AddDbContext(options => options.UseNpgsql()); ``` works in a regular...