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

Fixed issues #1, #3 added some new features. #7

Closed

Conversation

sachinkumarsingh092
Copy link

Added a sticker to welcome new users(see issue #3).
Added 'unknown' handler to reply to unknown commands.
Fixed error in 'empty_message' handle dispatcher.
Fixed errors to handle exception in 'welcome' and 'empty_message'.
Removed unnecessary comments and added meaningful ones in 'welcome' making code more concise(see issue #1).

Added 'unknown' handler to reply to unknown commands.
Fixed error in 'empty_message' handle dispatcher.
Fixed errors to handle exception in 'welcome' and 'empty_message'.
Removed unneceesary comments and added meaningful ones in 'welcome' making code more concise(see issue ILUGD#1).
@@ -82,17 +82,25 @@ def help(bot, update):
/github - link to ilugd github repos
''')

def unknown(bot, update):
Copy link
Member

Choose a reason for hiding this comment

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

nice

ilugd.py Outdated
except KeyError:
member_first_name = message.new_chat_members[0]['first_name']

phrases = ['Hello {}! Welcome to {} .Please introduce yourself. Dont forget to read the chat rules using !rules command'.format(member_first_name,message.chat.title)]
Copy link
Member

Choose a reason for hiding this comment

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

why modify the list it was planned to expand it so that if multiple people join on same day its too boring to receive same message

Copy link
Author

Choose a reason for hiding this comment

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

There was no documentation or comment about it. I will write a comment and add a few more welcoming messages. Should I proceed?

Copy link
Member

Choose a reason for hiding this comment

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

applies to same sticker what do you think (i was thinking like a list and throw a random one to chat)

Copy link
Member

Choose a reason for hiding this comment

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

There was no documentation or comment about it. I will write a comment and add a few more welcoming messages. Should I proceed?

yupz (also for stickers (I love gifs))

Copy link
Author

Choose a reason for hiding this comment

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

It sounds awesome! Also what about the '!rules' command mentioned in 'phrases'? How should I go about it?

Copy link
Member

Choose a reason for hiding this comment

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

ping @nk521 for more info (@nk521 we can merge both ?)
it's from a different bot

Copy link
Author

Choose a reason for hiding this comment

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

So, for the time being, I will make changes for stickers and 'phrases' and will raise an issue about the 'rules' command to be solved in future commits. Is that all right?

This reverts commit 9be38aa.

Added sections for more phrases, stickers and gifs to be added.
… be added.""

This reverts commit 9f3892c.

Added sections for more phrases, stickers and gifs to be added.
@sachinkumarsingh092
Copy link
Author

I messed up a bit. Please find the latest changes to be merged. Sorry about that.

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