| 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
|
|
|
|
| |
I missed this file in commit 6202a7c.
|
| |
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
This line was wrongly indented after the merge conflict resolution in
commit e86e05e.
|
| |
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This flag overrides detction and uses the specified profile path. This
helps when a non-default profile is used for Slack. It also helps if
autodetection fails; accordingly, recommend use of the flag in that
case.
Signed-off-by: Robbie Harwood <rharwood@redhat.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
webappsstore.sqlite seems to be deprecated, and firefox is migrating to
per-page locations. The values inside the database can be
snappy-compressed, so handle decoding with a new dependency on
python-snappy. Suggested by Trygve Aaberge.
Signed-off-by: Robbie Harwood <rharwood@redhat.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Firefox's default profile can have several different name suffixes,
including ".default-release", the legacy ".default", and others for
different release channels. Rather than guessing, just parse
profiles.ini to determine the default path. Suggested by Trygve
Aaberge.
Link: https://support.mozilla.org/gl/questions/1264072
Link: https://support.mozilla.org/en-US/kb/understanding-depth-profile-installation
Signed-off-by: Robbie Harwood <rharwood@redhat.com>
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
This makes the code much easier to read in my opinion. It also fixes
some errors about potentially unbound variables.
|
| |
| |
| |
| |
| |
| |
| | |
Although this file exists in the Chromium profile directory, it's empty.
I would assume Chrome/Chromium always uses the LevelDB file, so there's
no point in checking the webappsstore.sqlite file. Doing this allows us
to refactor some code in the next commit.
|
| |
| |
| |
| |
| |
| | |
Chromium encrypted values start with v10, while Chrome values start with
v11. I don't see any reason to check that the prefix is either of these
really, just remove it unconditionally.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
I see no value of having this option, and it's just confusing since the
script is meant for extracting the token.
|
| |
| |
| |
| | |
The script didn't work anymore for firefox after commit c7868cc.
|
| | |
|
| | |
|