Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API throwing exception "JSON integer is too large or small for int32" for item_id #57

Open
lowderjh opened this issue Jul 7, 2022 · 8 comments

Comments

@lowderjh
Copy link

lowderjh commented Jul 7, 2022

item_id is an int32 and needs to be updated to long

@mattf373
Copy link

mattf373 commented Jul 7, 2022

JSON integer too large issue for int32

A comment was added on June 30th 2022 about adding a PR for this stackoverflow - json int issue

Inquiring to see when this fix may get pushed out? Yes, we could fork the code and fix internally but will cause a lot of work to refactor the code & build pipelines as well as validate and test. If it's weeks or days then we could possibly wait for the push otherwise we will have to make the change on our end.

@lowderjh
Copy link
Author

lowderjh commented Jul 8, 2022

looks like a pull request has been added for this issue, hopefully it can be be reviewed and merged soon.

@Ldatz
Copy link

Ldatz commented Jul 8, 2022

Hi All, when this fix is merged, can someone tell me the best way to install this into my VS 2022 project. Do I just update the Podio.Async NUGet package manager or do I have to create my own Github version, make changes, and package myself?

@Ldatz
Copy link

Ldatz commented Jul 9, 2022

Hi All, since I'm not familiar with creating packages, I installed Malte.Podio.Async into my VS 2022 project and it works. I had to change some ints to longs in my code but, success. I'd would like to switch back to this version because I am guessing Pavlo has some affiliation with Podio. Does anyone know when there might be a new release of Podio.Async or if there is any issues using an NuGet plugin from an unknown source?

@GuidoKuth
Copy link

Same issue here. It would be very kind if the pull equest mentiones above could be merged into the project.

@madslund
Copy link

Any update on this? The pull request has been ready for over a week.
Could someone kindly merge and publish an updated package?

@keisenb
Copy link

keisenb commented Aug 15, 2022

Hey, the work here is already done, please approve the changes and merge this in. This is a huge bug and makes the repo pretty pointless right now since I can't interact with my items. Thanks!

@michaelmarziani
Copy link

Keep in mind this repo is archived and marked No maintenance intended. The PR I submitted is purely speculative. It may some day be merged and re-released, or not.

In the meantime I recommend this package, created by another collaborator, which has all the fixes in my PR: https://www.nuget.org/packages/Tvermose.Podio.Async

Good luck!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants