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.

Sign up for GitHub

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

Jump to bottom

Bulk transfer crashes the bot #5992

Closed
pogarek opened this issue Apr 3, 2017 · 9 comments
Closed

Bulk transfer crashes the bot #5992

pogarek opened this issue Apr 3, 2017 · 9 comments

Comments

@pogarek
Copy link
Contributor

pogarek commented Apr 3, 2017

@walaoaaa1234 check comments in #5988 .
If this won't get fixed we will need to figure out how to remove/rollback #5988 and #5987 from dev branch....
@walaoaaa1234
Copy link
Contributor

maybe limit it to 50 pokemons at a time

@pogarek
Copy link
Contributor Author

pogarek commented Apr 3, 2017

how ? can you code to have the bulk transfer optional and the classic one as default ?
We cannot merge dev to master till it will be resolved..

@Jcolomar would it help for you ? (the limit to 50)

@pogarek
Copy link
Contributor Author

pogarek commented Apr 3, 2017

@walaoaaa1234 will you be able to fix it or should I revert 437c2bd#diff-40e7f769ac13a8c2681747b100bfa53c and 70df546#diff-40e7f769ac13a8c2681747b100bfa53c ?

@walaoaaa1234
Copy link
Contributor

i am working on it

@pogarek
Copy link
Contributor Author

pogarek commented Apr 3, 2017

great , thnx

@Jcolomar
Copy link
Contributor

Jcolomar commented Apr 3, 2017

@pogarek didnt work to me. No transfer.

image

@walaoaaa1234 walaoaaa1234 mentioned this issue Apr 4, 2017
Merged
1 task
@MerlionRock
Copy link
Contributor

@walaoaaa1234 See comments I have for your latest PR

@MerlionRock
Copy link
Contributor

@pogarek say he can't get it working.

Anyone else facing same issue. I would love to merge all changes to Master soon.

@Jcolomar
Copy link
Contributor

Fixed #5998

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pogarek @walaoaaa1234 @MerlionRock @Jcolomar