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

Add Upgrade Pokemon Levelling Up #1452

Merged
merged 9 commits into from Jul 30, 2016
Merged

Add Upgrade Pokemon Levelling Up #1452

merged 9 commits into from Jul 30, 2016

Conversation

ghost
Copy link

@ghost ghost commented Jul 30, 2016

Fixed old PR

@ghost
Copy link
Author

ghost commented Jul 30, 2016

Code is complete feel free to make changes are any changes you want me to do

@mich3lam
Copy link
Contributor

Error CS0136 A local or parameter named 'dgdfs' cannot be declared in this scope because that name is used in an enclosing local scope to define a local or parameter

@mich3lam
Copy link
Contributor

nvm, issue was on my side. fixed, compiles fine

@MonsterMMORPG
Copy link

my upgrade logic easier and working. only what it lacks is i can not determine whether pokemon is already max upgraded or not : https://github.com/MonsterMMORPG/Pokemon-GO-Rare-Pokemon-Hunting-Farming-Bot/

@ghost
Copy link
Author

ghost commented Jul 30, 2016

Renamed variables as were confusing fixed now

@DeadElle
Copy link
Contributor

I always done this "by hand" but for lazy people this is awesome 👍

@MonsterMMORPG
Copy link

@cawk i didnt see you also check for whether pokemon is already max level or not?

@ghost
Copy link
Author

ghost commented Jul 30, 2016

last commit fixes bug with min>max and matches the naming convention in the tasks

@Newgamer1
Copy link

are you able to use the bot?

@AriSrijaya
Copy link

please update release 3.3 to 3.4

I have problem with sniping
no pokemon

@NecronomiconCoding NecronomiconCoding merged commit 7f3b649 into NecronomiconCoding:master Jul 30, 2016
schnapster pushed a commit to schnapster/NecroBot that referenced this pull request Dec 8, 2016
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

Successfully merging this pull request may close these issues.

6 participants