Shane Krueger

> I will try modifying the GitHub Action workflow to exclude the vsix install and see how that goes. The GitHub Action workflow builds the documentation project just fine without...

@sdanyliv is there any update/plan on supporting savepoints within transactions? I believe they are supported by SQL Server, Oracle, Postgres, MySQL, SQLite, and others. Many of them are implemented within...

See: - https://learn.microsoft.com/en-us/dotnet/api/microsoft.data.sqlclient.sqltransaction.save?view=sqlclient-dotnet-standard-5.1 - https://learn.microsoft.com/en-us/dotnet/api/system.data.sqlclient.sqltransaction.save?view=dotnet-plat-ext-7.0 - https://www.npgsql.org/doc/api/Npgsql.NpgsqlTransaction.html - https://dev.mysql.com/doc/dev/connector-net/latest/api/data_api/MySql.Data.MySqlClient.MySqlTransaction.html - https://docs.oracle.com/en/database/oracle/oracle-database/23/odpnt/TransactionSave.html#GUID-85B498C1-87F9-4A69-BD8E-2DE468400E3B - https://dev.mysql.com/doc/dev/connector-net/latest/api/data_api/MySql.Data.MySqlClient.MySqlTransaction.html - no Save support - https://dev.mysql.com/doc/refman/8.0/en/savepoint.html

This is actually an [input error](https://graphql-dotnet.github.io/docs/getting-started/errors/#input-errors), due to the fact that `IdGraphType` cannot parse a list. See here: - https://github.com/graphql-dotnet/graphql-dotnet/blob/49f24b574380f92f6c74f7963b8320167004438a/src/GraphQL/Types/Scalars/IdGraphType.cs#L47-L62 So `ParseValue` runs `ThrowValueConversionError`, which throws the exception you showed...

@BottlecapDave I looked into this further; you are correct, although `ParseValue` throws an error, `ValidationContext.cs:311` rethrows a generic error which should not contain C#-specific information.

Return null instead of an `IDataLoaderResult` like this: ```cs IDataLoaderResult? MyLoader(Key? k) { if (k == null) return null; // various code return loader.LoadAsync(key); } ``` The execution result see...

Just FYI, since `IDataLoaderResult` includes a method that returns `Task`, and as `Task` isn't `Task`, we can't make `IDataLoaderResult` into `IDataLoaderResult`. It's one of the most annoying parts about `Task`...

> `LoadAsync` returns an `IDataLoaderResult` where T may be null because nullable reference types aren't enabled for graphql-dotnet (Or at least, I believe this is the reason) NRT annotations are...

By the way, intenally `LoadAsync` already contains this code (although TKey is marked as notnull)  And there's this:  We could expose `DefaultValue` to be public ... but ideally...

Ok