GraphRequest property names break VB6

[nozzlegear]

From #1051:

I'm sure I'm the last visual basic programmer on the planet, but now this class is crashing as VB can't handle there being a query property and a Query property.

[nozzlegear]

@dg107 Can you confirm, is it just the GraphRequest class that breaks in Visual Basic because of the duplicate property names? Would you be open to adding a compiler constant to your build that disables the duplicate properties in that class?

I'm thinking about doing something like this, temporarily:

public class GraphRequest
{
    #if !SHOPIFYSHARP_VB_MODE
    [Obsolete("This property is deprecated, use " + nameof(Query) + " instead.")]
    public string query { get; set; }
    #endif

    public string Query { get; set; }
}

Then you could set SHOPIFYSHARP_VB_MODE=true inside your vbproj file:

<PropertyGroup>
    <DefineConstants>SHOPIFYSHARP_VB_MODE=True</DefineConstants>
</PropertyGroup>

Does that sound feasible to you? It'd only be a temporary solution, I plan on removing the deprecated properties in about 90 days.