| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
|
|
| |
When you're invited to a new channel, it's useful that it shows up in
the hotlist so you notice it. Previously it would not do that until a
message was posted to the channel after you joined. Now the join message
itself will also be added to the hotlist.
|
|
|
|
|
|
| |
If someone invited you to a private channel, it wouldn't work because
the channel would be unknown to wee-slack and the join function didn't
create a new channel instance.
|
|
|
|
|
|
|
| |
In commit 6478c14 the block rendering was changed to escape `<` and `>`.
However, this caused links and refs in normal (not rich_text) blocks to
be shown as refs instead of rendered correctly. The correct behavior is
to not process refs in rich_text blocks, but do it in normal blocks.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Commit 74da30342e8d328fc085d2158e37afda3b908f2c added support for using
rich_text blocks to format messages. With this change, the use of
replace_string_with_emoji() for text type "emoji" can result in running
the replacement twice as replace_string_with_emoji() is also called as
one of the last steps in SlackMessage.render().
This works fine in the default case when render_emoji_as_string = "",
however render_emoji_as_string = "both" results in output like:
π (π (:smile:))
instead of the intended:
π (:smile:)
This excludes blocks rendered earlier in the method from the second
call of replace_string_with_emoji().
|
|
|
|
| |
Slack began to require this for xoxc tokens today.
|
|
|
|
| |
Fixes #900
|
|
|
|
|
|
|
| |
This fixes a regression in commit 74da303 which caused < and > to be
removed from messages. Escaping them in unfurl_block_element so they can
be unescaped later isn't the nicest fix, but it's the easiest so I'll do
it this way until this is rewritten in the new version.
|
|
|
|
|
| |
After using blocks to render messages in commit 74da303, huddle
messages got the text twice since they already used blocks.
|
| |
|
|
|
|
|
|
|
|
|
| |
This caused some nicks and bots to be colored as offline if the option
weechat.look.color_nick_offline was enabled because not all nicks and
bots are included in the nicklist. Revert this for now, and see if we
can bring it back later.
This reverts commit 7310fe88d6fbf94d1bda7c10f24fa9e396d80311.
|
|
|
|
|
| |
This option wasn't accounted for when the message rendering was changed
to using blocks in commit 74da303.
|
| |
|
|
|
|
| |
Fixes #550 properly.
|
|
|
|
|
|
|
| |
When a Slack message is linked and unfurled (previewed) the footer
previously contained the channel name for the message, but now it just
says "Slack Conversation" or "Thread in Slack Conversation" instead.
This brings back display of the channel name.
|
|
|
|
|
|
|
|
| |
The commit 74da303 introduced a regression with messages not being shown
as deleted anymore because deleting only changed the text field, and
that's not being used anymore with that commit.
Fixes #897
|
|
|
|
|
| |
For some reason these errors were not reported by flake8 before version
6.1.0.
|
|
|
|
|
| |
It can be difficult to find a specific channel if there are thousands
of them in your slack workspace. This enhancement allows you to
filter the channels based on a regular expression.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of just using the text field of a message, use the rich_text
blocks to format the message.
The main reason for doing this was that messages that include images
suddenly started just having "This message contains interactive
elements." in the text field.
There are some other benefits as well:
- It fixes some bugs with multiple formatting styles not being applied
correctly.
- It fixes links in code blocks not being rendered verbatim (but with
text and url, like outside code blocks).
- It fixes code blocks sometimes having newlines after/before ``` and
sometimes not, depending on how the message was sent.
It looks like messages with images are now back to having the proper
text in the text field (for new messages), so this change isn't
necessary to fix that after all (except for for messages in the history
sent in the last couple of days), but since it fixes other issues as
well, it's best to keep it.
There is one bug left when inline code blocks contains links. Then there
will be ` characters before and after the link, which shouldn't be
there, but I'll leave fixing this for later.
Fixes #354, fixes #550, fixes #893
|
|
|
|
| |
Not sure if this ever occurs, but better safe than sorry.
|
|
|
|
| |
Fixes #888
|
|
|
|
| |
Fixes #889
|
| |
|
|
|
|
|
|
| |
Automatically set completion_default_template on wee-slack buffers, so
emojis and usergroups can be tab completed without having to set
weechat.completion.default_template manually.
|
|
|
|
|
|
|
|
|
| |
If the line isn't found, line_pointer becomes an empty string which will
crash WeeChat when it's used with hdata_update. This happened when the
history for a channel that contained a reply also posted to the channel
was loaded, because in that case wee-slack tries to modify the parent
message before the channel history is printed, so it couldn't find the
line.
|
|
|
|
|
|
|
|
|
|
| |
Since WeeChat now has multiline support, the problems with using
prefix_nick_<color> (mainly that it isn't possible to distinguish
between a multiline message and multiple messages when
weechat.look.prefix_same_nick is set) isn't present anymore, so we can
use this tag to properly mark lines as chat messages from a nick. This
also means that we don't have to use prefix_same_nick or
nick_prefix/suffix ourselves anymore.
|
|
|
|
|
|
|
|
|
| |
Since WeeChat now has multiline support, the whole message is in one
chat line. This means that we don't need to look for all the lines
belonging to a message and modify them, we can just modify the line for
the message directly. This has the benefit of making it possible to
increase and decrease the number of lines for a message, rather than
having to join extra lines or get blank lines.
|
|
|
|
|
|
| |
Don't insert a prefix at the start of each line of a message so it
renders normally in WeeChat 4.0.0 which supports chat lines with
multiple lines.
|
|
|
|
|
|
|
|
|
| |
Without spaces, WeeChat will consider all the nicks as one word, which
means that rather than wrapping between nicks, it will fill the whole
line and wrap in the middle of nicks. Additionally, if you have disabled
weechat.look.align_multiline_words it will start from the beginning of
the line when wrapping, rather than after the prefix, which doesn't look
nice.
|
|
|
|
|
| |
Remove a space before a colon and add a comma after a sentence starting
with if.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The url_private link for Slack posts just downloads a json of the post,
which isn't very useful to click on. The permalink goes to the web view
for a post, which is what we want. The web client adds origin_team and
origin_channel, which makes the post page show info about where the post
is shared, so add that here as well.
We still want to use url_private for images etc., not permalink there as
well, because that links to the Slack client with the image shown in the
sidebar, which is much more inconvenient than just getting the image
directly.
|
|
|
|
|
|
|
|
| |
If a link preview was added by an app, like GitHub, instead of directly
by Slack, it was not detected by wee-slack as a link preview, so it
would be shown even if link_previews was set to false.
Fixes #834
|
|
|
|
|
|
|
|
|
|
|
| |
One user experiences receiving a "HTTP/1.1 200 Connection established"
header block before a "HTTP/2 200" header block with the headers. I'm
not sure why specifically this happens, but I know a similar response
will happen if redirects are followed.
This wasn't supported by the http_check_ratelimited function which
splits the header from the body. It now splits out all the header blocks
from the body and uses the status and headers from the last one.
|
|
|
|
| |
Fixes #871
|
|
|
|
|
| |
This is only required for WeeChat < 1.5, and we require >= 2.2, so
there's no point in keeping this.
|
|
|
|
|
|
|
| |
After moving the token in Slack API requests from a query string param
to an Authorization header in commit 2185a58 we require WeeChat >= 2.2
because that was the version which added support for setting HTTP
headers.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We used the response from rtm.start to find which IMs and MPIMs to open,
but after moving away from that we didn't have the information from the
response from conversations.list, so commit 962bbc5 changed to only open
unread ones.
However, it turns out that conversations.info has the information, and
we are already requesting that for each IM/MPIM to check if it's unread,
so we can use that to open the correct ones.
Fixes #875
|
|
|
|
|
|
|
|
| |
Old type MPIMs (has `is_group: true`) has `is_open`, but not
`unread_count_display` in the response from `conversations.list`, so
they were not marked as unread on start if background_load_all_history
was set to false because we have to request the history, but
`check_should_open` created the buffer, so it wasn't requested here.
|
|
|
|
|
|
|
|
|
|
| |
Apparently there are some bots which are not listed in users.list and
which sends messages where the only info about the sender is a bot_id.
To fix this properly we will have to look up the bot with bots.info when
we encounter an unknown bot_id, but for now just fall back to using the
id directly like we do with unknown users to make it possible to use
triggers and filters.
|
|
|
|
|
| |
I've been the maintainer of this script for several years now, and Ryan
isn't involved anymore, so it makes sense to update the script author.
|
|
|
|
|
| |
The script will use the new methods if the WeeChat version is new
enough, and the old deprecated methods if it's too old for the new ones.
|
| |
|
| |
|
| |
|