GraphQL fluent query builders in 6.26.0 - joy, relief and a some questions!

[lanthonyneville]

Firstly, @nozzlegear - thank you, thank you, thank you! What a piece of work!! I was hand writing text GraphQL queries (couldn't get the schema file working in VS) ... constant flicking back & forth between the Shopify documentation & my code, getting it wrong 20 times, tracing the request & response with Fiddler, hair loss, tears etc. What a difference this makes!!! And you retained compatibility with .NET Fmk ... sob!

Also acknowledgement obviously due to @charlesdevandiere.

Now the questions:

1. I am using the ProductSetOperationQueryBuilder with ProductSetInput as it is the recommended way to "upsert" a product. There should be a nullable "id" property on ProductSetInput - set to identify an existing product, null for a new product, but your class does not have this. Omission or some other way to provide the ID?
2. I am using ProductQueryBuilder & ProductVariantQueryBuilder to pull products ... can't see how to set the arguments (the required ID) however ... there's no Arguments property on the builder as indicated in your notes. Some other way to do this?

Thanks again

[nozzlegear]

Thanks for the kind words @lanthonyneville! And yes, big thank you and shout out to @charlesdevandiere as well for the inspiration from his project graphql-query-builder-dotnet – Charles, I've made sure to properly credit you for your work in the attributions file, but again, thank you!

[nozzlegear]

@lanthonyneville To answer your questions:

1. I had to look at Shopify's graphql schema for this one, but I believe that ProductSetInput is correct and shouldn't have an id property. Instead, we're supposed to pass in the id as an argument:

var builder = new ProductSetOperationQueryBuilder()
  .SetArguments(args =>
    args.Input(new ProductSetInput
    {
      title = "Updated Product Title",
      // ...
    })
    .Synchronous(true);

    if (yourProductId != null)
    {
      args.Identifier(new ProductSetIdentifiers
      {
        id = yourProductId
      });
    }
  )
  .Product(p => p
    .Id()
    .Title()
    .Handle()
    .UpdatedAt()
  )
  .UserErrors(e => e
    .Field()
    .Message()
  );

(Note: this code uses the new .SetArguments fluent argument builder that I'm adding today in #1240.)

2. When I published ShopifySharp 6.26.0 with the new query builder support, I accidentally left some code unimplemented that was supposed to check whether a type actually supports arguments or not. Instead that code only returned true for types that are operations, which is why the top-level query and mutation types all have argument builders, but none of their sub-query types do. I'm fixing that right now, also in Add fluent argument builder method and generate argument builders for all types that support them #1240, and will publish today as well.

[lanthonyneville]

Oops, yes you are right about ProductSetInput and the identifier argument ... it used to have an "id" property but now it is deprecated.
Thanks again, and looking forward to trying out the next update.