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

feat: bot integration with group chat #364

Draft
wants to merge 32 commits into
base: main
Choose a base branch
from

Conversation

agazso
Copy link
Contributor

@agazso agazso commented Sep 8, 2023

Added group chat support.

Known issues and limitations:

  • The bot only respond to messages which start with @${BOT_NAME} in a group chat, otherwise it could end up with an infinite conversation when there are multiple bots. For similar reasons bots also ignore their own messages in a group chat.
  • Because of this, the history also only includes messages which are addressed to the bot, therefore the bots will have no context of a group chat conversation in general.
  • This is done: Currently the user address (which is an ethereum address) is used as the user name. I saw the bot responding to me sometimes with my @address. It is possible to look up the names based on the address and fill that in for the requests but this is not done yet.
  • The chat rooms are persisted on the waku network, so after restarting the bot it will remember the groups. However currently it does not check historical invites, so any invites when the bot is offline are lost. This is something we can add later.

@vercel
Copy link

vercel bot commented Sep 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
waku-objects-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 10, 2023 7:54pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

1 participant