-
Notifications
You must be signed in to change notification settings - Fork 3
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
create a discord bot #6
Comments
Working on it |
Hi @rHuggler, I can also help with it. :) |
Hi @ParthS007, help would be appreciated. I've started working on it this evening and didn't get that far. https://github.com/rHuggler/krorg-bot It works, but it doesn't look very good (to me) and still doesn't have any validation. |
@rHuggler ping me on discord when it's ready to be moved in to the org! |
I'm thinking about changing to Javascript, the existing Discord API wrappers for Python I've found are outdated and/or have vulnerable versions of other packages as sub-dependencies. Any thoughts on that? |
Hey, I feel like that would be okay to do. Do you have any documentation to review? If anyone else has any ideas on how to create the bot with python, drop a comment and provide some examples on how to implement. |
JS sounds fine! as long as it works :) |
@kenneth-reitz Is the Bot only for adding a user to Org? |
we can have it do github notifications too |
Yes @kenneth-reitz, We can do the notifications part by adding webhook of the bot and selecting the events that are to be shown as notifications on Discord. |
Wrapping up the day and it's working now 🎉 I've ran some tests on a private server and a test organization I've created and I was able to add an user. There's still a lot of work to do (input validation, Github response handling, etc.) but it works :) |
yay!!!! |
I need some help figuring out these things:
|
@rHuggler, We can deploy it on Heroku and we have to enable the worker instance which will keep it running. |
Heroku works, or glitch. |
Make a Heroku app and deploy it, and then we'll transfer the app to my org. Let's do autodeploys from GitHub too. |
tests can just be "does this run" |
Alright. It would be good if someone could collaborate as well, I won't be available today |
@rHuggler I will be happy to help. :) |
@rHuggler @ParthS007 I'll be happy to collaborate too 😃 , let me know if there's anything I can help with |
I'll add you as collaborators when I'm back from lunch EDIT: Done, you're both invited to be Collaborators. To everyone else, feel free to fork and open a Pull Request! |
thanks @rHuggler! |
Hi there, |
No description provided.
The text was updated successfully, but these errors were encountered: