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

WIP: Discord notifications #240

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

thiemok
Copy link

@thiemok thiemok commented Aug 16, 2016

I implemented a discord bot to report rare sightings.
Originally based on this.
Its reporting a pokemon's name, number location and time until despawn, as well as an icon for easier recognition.

TODO:

  • I will some functions based on the report data, like where to look for an specific pokemon.
  • It needs python 3.4.1+ to work unfortunately I don't know how to keep compatibility with python 2.7 in the requirements.txt, however it does run individually, so pip install is the only thing that breaks
  • Message translations are needed
  • Readme needs an update
  • The watchlists persistence is implement using a simple json file, this needs some value checks and maybe it should be saved in the db

@SapitoSucio
Copy link

Can help you with Spanish translation

@thiemok
Copy link
Author

thiemok commented Aug 16, 2016

@DavidSelem great, maybe wait until i have added the additional commands. But you can start by adding spanish translation to the main project, since that does not have spanish translation as well.

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