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

Feature Enhancement: Snip request through telegram #5877

Merged
merged 6 commits into from
Jan 17, 2017
Merged

Feature Enhancement: Snip request through telegram #5877

merged 6 commits into from
Jan 17, 2017

Conversation

MerlionRock
Copy link
Contributor

Re-Created due to conflicts

Added new feature:
Snipping through telegram, enable it in config, Sniper Task "mode": "telegram"

Also fixed the problem of not able to find Pokemons without EncounterID and SpawnPointID.

Fixes#5863

@mention-bot
Copy link

@MerlionRock, thanks for your PR! By analyzing the history of the files in this pull request, we identified @YvesHenri, @Gobberwart and @javajohnHub to be potential reviewers.

Make changes to ignore Pokemon in snipe catch list for telegram mode.
To prevent looping errors due to wrong input by users, try except expression added to telegram snipe mode
@@ -406,12 +449,15 @@ def work(self):
else:
self.bot.sniper_disabled_global_warning = False
targets = []
target = {}
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems nowhere to use 'target' in the following code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Noted. Will delete it off. I changed this files too many times and lost track what I did.

Copy link
Contributor

Choose a reason for hiding this comment

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

Got it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done :)

Traget = {} removed
Added instructions on how to activate and use manual snipping through telegram
@solderzzc
Copy link
Contributor

solderzzc commented Jan 17, 2017

👍

Approved with PullApprove

@solderzzc solderzzc merged commit ef91376 into PokemonGoF:dev Jan 17, 2017
solderzzc added a commit that referenced this pull request Jan 17, 2017
Feature Enhancement: Snip request through telegram (#5877)
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.

4 participants