| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Without any arguments, /rehistory will now just clear the buffer and
print the history again. With -remote, the history will be downloaded
from Slack and processed again.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Fixes #775
|
| |
|
| |
|
|
|
|
| |
This isn't necessary anymore.
|
|
|
|
|
| |
This fixes typing indicators for mpdms, which was previously not
working.
|
|
|
|
|
|
| |
users.info takes the id, not the name of the user. This worked in
practice before this change because it was only called for unknown
users, and unknown users gets the name set to the id.
|
|
|
|
|
|
|
| |
A few places used slack_name when displaying the name to the user. name
should be used instead because tincludes the correct prefix. The first
place was just for dms though which doesn't have a prefix, so that won't
show any difference.
|
|
|
|
|
|
|
|
| |
We shouldn't change the full name for typing indicators because autosort
will reorder the buffer, and it may break scripts using the full name.
We didn't show typing indicator until the previous commit anyways,
because the full name was never updated until that.
|
| |
|
|
|
|
|
| |
formatted_name checks if colors should be enabled, so it's not necessary
to send it in as a parameter as well.
|
|
|
|
|
| |
There was some mechanism for this, but it wasn't used. Replaced the old
unused mechanism with a new one.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
I mainly did this because I'm going to call formatted_name with
style=default before team is set, and that didn't work as it was because
the other styles used the team.
For SlackDMChannel I moved colorizing into the branch for the sidebar
style because colors should never be included in the other styles.
formatted_name is never called with enable_color=True and another style
than sidebar, so the behavior doesn't change.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Slack only sends typing notices every 5-6 seconds, so when we expired
them after 4, a user which continously typed a message would disappear
from the typing bar every 4 seconds and appear again after a second. Now
they are shown continously, and for about the same duration as the web
client.
|
|
|
|
|
| |
SlackChannel.members is overwritten in set_members and user_joined, so
it could become stale when it was an attribute.
|
|
|
|
|
| |
Lots of places used members instead of get_members, and it's
inconvenient having to call a function anyways.
|
|
|
|
| |
We don't receive any event when a user has stopped typing.
|
|
|
|
|
|
|
|
|
| |
Slack doesn't render the `text` property at all when `blocks` is
non-empty, so we shouldn't render both `text` and `blocks`. However,
because we don't support the `rich_text` block type yet, we have to fall
back to the `text` property when `rich_text` is present. Therefore,
render both `text` and `blocks` if `rich_text` is present, else render
only `blocks` if it is non-empty, and only `text` if `blocks` is empty.
|
|
|
|
|
| |
I don't think this exists anymore, and we support rendering markdown so
we shouldn't use fallback anyways.
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, files with a mode other than "tombstone" were assumed to have a
"title" and "private_url". This is not necessarily the case; in particular,
files with the "hidden_by_limit" (removed from visibility because the
workspace has exceeded its free-tier storage limit) mode have *only* "id" and
"mode". This resulted in a KeyError trying to look up the "title" and
"private_url", and truncated history at the message with that file.
This commit explicitly handles "hidden_by_limit" files, and makes the handling
of any other unknown modes more robust by explicitly checking for "title" and
"private_url" before rendering a message with them.
|
| |
|
| |
|
|
|
|
| |
Fixes #764
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Slack now disallows # in OAuth redirect uris, so we can't use it to
prevent GitHub Pages from seeing the OAuth code anymore.
Since the OAuth code is only valid once and expire after 10 minutes, I
don't think it's a problem that they are exposed to GitHub Pages, so
this is the new default so the same registration process can be kept.
However, for people worried about this, there's a new -nothirdparty
option to /slack register which can be used which doesn't expose the
code.
Fixes #766
|
|
|
|
| |
As of today, these can no longer be created.
|
| |
|
| |
|
|
|
|
|
| |
For deleted message events, this field may not exist (e.g. when a giphy
preview is deleted).
|
| |
|
|
|
|
|
|
|
|
|
| |
/reply takes a message, so default completion should work there. /slack
talk only takes one argument with a comma separated list of nicks, so
technically preventing default completion is correct for that. However,
until we support completing multiple nicks separated by comma,
preventing completion of nicks as additional arguments makes it
inconvenient to use.
|
|
|
|
|
| |
The replies property doesn't exist anymore, it has been replaced by
reply_users and reply_users_count.
|
| |
|
|
|
|
| |
Relates to #767
|
|
|
|
|
|
|
|
|
|
|
| |
Completion for our commands can't use || because they can be used as
subcommands for /slack and weechat doesn't support multiple templates
with the same subcommand. See https://github.com/weechat/weechat/issues/1481
This makes the completion for /reply slightly incorrect. It now allows
you to complete two thread arguments when not using -alsochannel. To fix
that, we would have to write a custom completion function. It's not
hard, but I'm not sure it's worth the effort for such a small detail.
|
|
|
|
| |
This bug was introduced in commit b2618af.
|
|
|
|
| |
This was broken in the previous commit, be5d890.
|
|
|
|
|
|
| |
Previously, the emoji completion only returned emojis if you tried to
complete after typing a :. This prevented it to be used in commands
without typing : first.
|
| |
|
|
|
|
| |
Fixes #728
|
|
|
|
| |
Relates to #728
|
| |
|
| |
|