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

Eliminate error caused by typo #5954

Merged
merged 1 commit into from
Mar 6, 2017
Merged

Eliminate error caused by typo #5954

merged 1 commit into from
Mar 6, 2017

Conversation

xty
Copy link
Contributor

@xty xty commented Mar 6, 2017

Currently when "pinapberries" is specified in the item list under UpdateLiveInventory in config.json, the following exception is raised:

File "./pokemongo_bot/cell_workers/update_live_inventory.py", line 200, in get_item
    raise ConfigException("item '{}' isn't available for displaying".format(item))
pokemongo_bot.tree_config_builder.ConfigException: item 'pinapberries' isn't available for displaying

It turns out "pinapberries" is misspelled as "pinabberries"; this update serves to eliminate the typo.

Currently when "pinapberries" is specified in the item list under UpdateLiveInventory, the following exception is raised:
File "./pokemongo_bot/cell_workers/update_live_inventory.py", line 200, in get_item
    raise ConfigException("item '{}' isn't available for displaying".format(item))
pokemongo_bot.tree_config_builder.ConfigException: item 'pinapberries' isn't available for displaying

It turns out "pinapberries" is misspelled as "pinabberries" and this update serves to eliminate the typo.
@mention-bot
Copy link

@xiatianyu, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeXtroTip, @atxfun and @Arcasss to be potential reviewers.

@pogarek
Copy link
Contributor

pogarek commented Mar 6, 2017

👍

Approved with PullApprove

@pogarek pogarek merged commit b2ab903 into PokemonGoF:dev Mar 6, 2017
@pogarek
Copy link
Contributor

pogarek commented Mar 6, 2017

merged #5954

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.

3 participants