podio / podio-dotnet

Podio .NET client
MIT License
19 stars 39 forks source link

item_revision_id updated to long #47

Closed PavloBasiuk closed 6 years ago

PavloBasiuk commented 6 years ago

I'd recommend to build from master.

steveglen commented 6 years ago

@PavloBasiuk master appears to only contain an async API which is not the same as the Podio package https://www.nuget.org/packages/Podio/1.5.6

Can you confirm if this package is built from the v1 branch, and Podio.Async is built from the master branch?

Please ask the people updating status.podio.com to clarify what's changed to fix this issue, to help other consumers of the API to fix their apps.

Denacke commented 6 years ago

I built from the v1 branch with the changed int -> long and it worked as a drop in replacement for the nuget package in our solution.

steveglen commented 6 years ago

building from v1 and applying the same change also seems to work for me as a drop in replacement for the nuget Podio package.

PavloBasiuk commented 6 years ago

Thanks for confirming!