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

Workaround Empty Forts #2018

Closed
wants to merge 3 commits into from
Closed

Workaround Empty Forts #2018

wants to merge 3 commits into from

Conversation

loklam
Copy link

@loklam loklam commented Jul 31, 2016

Short Description: Sometimes, the API return empty forts. This causes move_to_fort fail. To workaround this, return prev forts, if API return empty forts.

Fixes:

@Sammy-L
Copy link

Sammy-L commented Aug 1, 2016

I've tested this code. It works very well. Niantic changed the search respond rate to about 4s since last patch. So the fort scan nearly always return empty result. Saving the old result does help for situations while server refuse to give response.

@kloz8879
Copy link

kloz8879 commented Aug 1, 2016

I Have also tested this and it works great! Thank you.
2016-08-01_07h15_08

@loklam
Copy link
Author

loklam commented Aug 2, 2016

This has been fixed at #2269

@loklam loklam closed this Aug 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants