`IncrementSet` should accept a `long` about algoliasearch-client-csharp

[sbellone]

Merged through #799, thanks again! 👍

[TomKlotzPro]

Hey @hognevevle,

Thanks for this feedback, I'll check with the engine-team if we can send a long.

I'll keep you updated about this.

[TomKlotzPro]

Hello @hognevevle,

I've checked with the engine team, and indeed, this should be a long! Do you want to open the PR to update the code ? I could do it but next week. 😞

However, I'm preparing a new minor release for next week, so it fits really well.

[hognevevle]

Opened #777 . Decided to add overloads for IncrementFrom/IncrementSet instead of updating the existing methods, to avoid a breaking change.

I'm not sure if it there is any use case for a long in Decrement/Increment, so for now I didn't touch those.

[TomKlotzPro]

Thank you 🙇
I'll take a look today, nice job 👍

[sbellone]

Hi @hognevevle, thanks for the PR!

but it still prevents me from using .ToUnixTimeMilliseconds() which I would prefer

Just note that you can still build the operation yourself:

var record = new Record
{
    ObjectID = "myID",
    Count = new PartialUpdateOperation<long>
    {
        Operation = PartialUpdateOperationType.IncrementFrom,
        Value = (long)2
    },
};

But it's indeed handy to have it built-in 👍