Skip to content

Fix XP calculation to handle None values and set default XP to 0#300

Open
Unlucky-Life wants to merge 3 commits intomainfrom
fix_for_mainpokemon_xp_issue
Open

Fix XP calculation to handle None values and set default XP to 0#300
Unlucky-Life wants to merge 3 commits intomainfrom
fix_for_mainpokemon_xp_issue

Conversation

@Unlucky-Life
Copy link
Collaborator

Fix for #297 of mainpokemon xp not being set.
Solution -

  1. Adapt the proper Fallbackvalues for MainPokemon Object that were None before.
  2. Adding a "or 0" statemtent and setting main_pokemon.xp to 0 if not set yet.

@MrElektronz
Copy link

I also fixed that issue in a PR of mine and didn't see it being fixed 2 weeks ago already :D
If your fix had been merged already I would not even have encountered that bug anymore ^^

Looks good to me, but I am unable to approve. Are contributors who have permission to do so aware of your PR and somewhat active?

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.

2 participants