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

Fix config.json.pokemons.example release config #980

Merged

Conversation

alvarogzp
Copy link
Contributor

Properties in the json were release_under_cp and release_under_iv, but code is searching for release_below_cp and release_below_iv, so this was not working if used by users.

@alvarogzp
Copy link
Contributor Author

Modified also evolve_all_worker to use release_below_cp and release_below_iv too.

There are no more places where release_under_cp and release_under_iv are used.

@alvarogzp
Copy link
Contributor Author

Same with cp_iv_logic to logic, as seen in #947

@douglascamata
Copy link
Member

Looks good to me.

@solderzzc
Copy link
Contributor

Thanks.

@solderzzc solderzzc merged commit 2634cc3 into PokemonGoF:dev Jul 26, 2016
@alvarogzp alvarogzp deleted the fix-config-example-release-pokemons branch July 26, 2016 16:22
MFizz pushed a commit to MFizz/PokemonGo-Bot that referenced this pull request Jul 29, 2016
* Replace 'release_under_cp' to 'release_below_cp', and 'release_under_iv' to 'release_below_iv' in config.json.pokemons.example to match what code searchs for

* Make evolve_all_worker use release_below_cp and release_below_iv too

* Replace cp_iv_logic to logic
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.

3 participants