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

IncubateEggs update #5125

Merged
merged 27 commits into from
Sep 4, 2016
Merged

Conversation

DeXtroTip
Copy link
Contributor

Short Description:

IncubateEggs task now return WorkerResult depending on egg hatching.
Now uses class Pokemon from inventory.py to display Pokemon stats and data.
Created table eggs_hatched_log in db to log eggs hatched.
Print ncp of the Pokemon.

Fixes/Resolves/Closes (please use correct syntax):

@DeXtroTip
Copy link
Contributor Author

CI failed because the incubate_eggs_test.py isn't updated with the changes I made so the task uses de DB. I don't know what I must change here.



class IncubateEggs(BaseTask):
class IncubateEggs(Datastore, BaseTask):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove Datastore

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I was waiting your PR to be merge, already had changes made. 👍

@solderzzc solderzzc merged commit 0e4c94c into PokemonGoF:dev Sep 4, 2016
@DeXtroTip DeXtroTip deleted the incubate_eggs_update branch September 4, 2016 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants