Shane Krueger

Results 854 comments of Shane Krueger

Looking at the code right now, it looks like 'custom containers' are supported so long as they implement `IList`, have a default constructor, and support adding items to the collection....

Yes, this PR has been merged for 8.x. It will bump GraphiQL to 3.2.0

The page is generated from this: https://github.com/graphql-dotnet/server/blob/master/src/Ui.GraphiQL/Internal/graphiql.cshtml Can you write a PR that includes the necessary changes to fix the page? Please note that the version provided by this middleware...

I'm very interested in this functionality as well. I've added a PR as a starting point, having Cline / Anthropic AI see if it could figure out how `client-preset` worked...

I wrote a working plugin for client-preset to replace the functionality in near-operation-file-preset. As a plugin, it supports all options that client-preset does, including persisted document support. Does anyone want...

I'd like to comment that in the main GraphQL.NET package we had an issue where dictionaries and classes that implemented list interfaces were identified as collection types. This was fixed...

> Replaces #3970. > > With this implementation, we can also use a legacy flag to deliver the new feature in v7 Let's just have a PR target `master`, then...

@gao-artur This PR is closed so I'm deleting the branch. You can always restore it if need be. Ditto with other old branches.

Looks pretty good. Couple comments: > Argument name should match `@key` name > Allow only arguments marked with `[FromServices]`, key and `CancellationToken` > Allow only arguments marked with `[FromServices]` and...