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

commands: Add /query support, to create a private chat with another user #338

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

Conversation

3v1n0
Copy link

@3v1n0 3v1n0 commented Dec 1, 2022

Support initiating chats with other users using /query command.

When inside a matrix channel or server user id can be in the form:

  • @user:server.ext
  • user:server.ext
  • @user (it looks for @user:current-server.ext)
  • user (it looks for @user:current-server.ext)

When inside a channel, can be just the nickname of the user, and we'll
figure out the corresponding id.

Re-initiating a left chat is supported, but may need some fixes.

3v1n0 added 2 commits December 1, 2022 02:19
Support initiating chats with other users using /query command.

When inside a matrix channel or server user id can be in the form:
 - @user:server.ext
 - user:server.ext
 - @user (it looks for @user:current-server.ext)
 - user (it looks for @user:current-server.ext)

When inside a channel, can be just the nickname of the user, and we'll
figure out the corresponding id.

Re-initiating a left chat is supported, but may need some fixes.
This is now easy enough to handle.
@strk
Copy link
Contributor

strk commented Dec 27, 2022

I've just tested this, works great, thanks @3v1n0 ! It would be great to get /names working next :)

@ilf
Copy link

ilf commented Feb 22, 2023

Thanks for this! It should fix #155 and #65

What's needed to get it merged?

@yourealwaysbe
Copy link

I found a bug in commands.py. line 1164 erases the original value of buffer so that lookups on other servers will fail. This causes a problem if you want to manage multiple matrix accounts.

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.

5 participants