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

Bluebird bug #160

Closed
iPaulis opened this issue Sep 20, 2016 · 2 comments
Closed

Bluebird bug #160

iPaulis opened this issue Sep 20, 2016 · 2 comments
Milestone

Comments

@iPaulis
Copy link

iPaulis commented Sep 20, 2016

Sometimes during a batch transfer an error message window appears like this:
bluebird bug
It does not happen always, but it happens often. It even happens with single transfers, but this time there is no error message window, only info in console, as shown here:
bluebird bug 2
This error can also happen during evolves such as here:
evolve bug 3
Bug reproduced in v1.5.2 and v1.5.3.

After the error is shown, all pending operations are stopped. Many times, just clicking transfer or evolve buttons makes the program start working again and complete the transfers or evolves left before the error happened, as they are still selected. However, there is always the possibility that the same error is thrown again.
I've never had any problem transferring before and if there was I didn't notice because all the selected pokemon were trasnfered anyway, so in practice is like there never was a error to begin with. Now we may be aware of the errors that are happening, but if that makes the transfers stop it doesn't seem very practical in my opinion.

@vinnymac vinnymac added this to the v1.5.4 milestone Sep 21, 2016
@vinnymac
Copy link
Owner

vinnymac commented Sep 21, 2016

I can easily prevent it from stopping transfer/evolve when an error happens. But that is a feature not a bug. The advantage is knowing what is failing. If we don't stop the transfer/evolve then we end up in bad situations where we lie and pretend everything worked when it may not have.

It is strange that single transfer/evolves wouldn't show the error message.

One thing I can do however is detect [INTERNAL] errors based on the error message, and then I could continue and just log it. I think that might be best in this case. After reading the stacktrace further it sounds like an error in building the envelope which looks bad. I think that this might be due to pogobuf's out of date dependencies. I'll think on this some more.

@iPaulis
Copy link
Author

iPaulis commented Sep 23, 2016

It works very nicely now in v1.5.4. The errors appear in console but the transfer process is not interrupted, and it is able to complete all the transfers.

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

No branches or pull requests

2 participants