search

JIITODC / bot

JIITODC's very own telegram bot!
MIT License
6 stars 10 forks source link

Add Basic Features to the Bot #2

Closed Sabreen-Parveen closed 4 years ago

Sabreen-Parveen commented 4 years ago

The following functionalities are added: -The bot introduces itself. -Greets when a user joins the chat -Says goodbye when a user leaves the chat -The group members can make a to do list.

weirdwiz commented 4 years ago

@JIITODC/bot is it merge ready? Where can this be hosted?

Ankurk99 commented 4 years ago

Do we need a goodbye message? Like how will the user see the message when they have already left the channel?

rexagod commented 4 years ago

Note that this PR invalidates #1 by shifting the default language from JavaScript to Python, which is not what the community agreed upon. Nevertheless, if folks want to work on a Python version, we can most definitely open up a separate branch for that.

Ankurk99 commented 4 years ago

As it was struck for a long time so we decided to start the work on it and the people interested in developing and maintaining the bot were more comfortable in using python, so we decided to go with it. But yeah we can definitely work on both the bots with different branch and they can have different functionalities.

DelusionalOptimist commented 4 years ago

Do we need a goodbye message? Like how will the user see the message when they have already left the channel?

The goodbye messages have been discarded. The changes will be pushed soon.

Note that this PR invalidates #1 by shifting the default language from JavaScript to Python, which is not what the community agreed upon. Nevertheless, if folks want to work on a Python version, we can most definitely open up a separate branch for that.

Working on a separate branch for python sounds good, we can surely do this :+1:.

DelusionalOptimist commented 4 years ago

@weirdwiz @Ankurk99 Some updates :-

After the above tasks are complete, the bot will be merge and deploy ready. From then on new issues will be worked upon. As decided, these changes will be merged into a separate python branch. Please make a new branch so that we can change the base branch for this PR.

weirdwiz commented 4 years ago

Thanks for the updates :D

rexagod commented 4 years ago

Please change the target branch to py (master -> py).

Sabreen-Parveen commented 4 years ago

Done!

DelusionalOptimist commented 4 years ago

@weirdwiz This can be merged now. :smile: