-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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] Configurable Pokemon Sniping #1274
Conversation
Tested and working great with default settings and SnipeAtPokestops turned on. |
wait. i didnt understand.. how does it know where pokemons are spawning since it requires new location every time .. so in order for it to work and rly snipe a pokemon you want you need to recompile everytime after u add a pokemon |
Magic. Just kidding, everyone knows sniper's scopes let you see things far away Pokevision |
@timelf123 useful comment |
how about magic |
Oh wow. I missed that part for some reason |
Really cool how you incorporated pokevision into the app to snipe pokemon you're looking for :D ill have to manually patch this in on my lunch break. Any issues so far with pokevision blocking your api calls after a while? |
None yet. I'd love to find a different source that we controlled a bit for locations, but the concept seems to be working fine so far. |
@kaeus why not to use data from #sniping channel on Discord? |
I want to rework the settings to be easier to manage so you have just a list of locations and a list of pokemon. |
@goodwind The #sniping channel doesn't have a standardized format yet. Might look into discord API since this is now in the main repo. |
Feature will work with the rest of the logic every 5 Pokestops to snipe configured areas for Pokemon. May need some updates for data calls sine I'm relying on an external API call to determine spawns.
Utilizes two new settings:
bool: SnipeAtPokestops
List: PokemonToSnipe
Example:
"PokemonToSnipe": [
{
"Locations": [
{
"Latitude": 38.556807486461118,
"Longitude": -121.2383794784546
}
],
"Pokemon": "Dratini"
}
]