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

Send group emoji #166

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

Conversation

MikeSperone
Copy link

In response to #78 and my own desire for such a feature, I added the ability to send the conversation emoji with the command of /l or /like . It will default to the thumbs up emoji, as facebook does, if there is no group emoji set.

This is working well on two of my mac computers, across various group and one-on-one chats. I was unable to get the tests running, so I didn't add a test for this. I saw another PR might have had a fix for the tests? I can add a proper test if that gets fixed.

I noticed a package-lock.json was added with my changes, I left it in there but I can remove it if you'd prefer.

This is my first git hub/open source Pull Request, so if I mixed something up with the Pull Request process, please let me know. Thank you.

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.

None yet

1 participant