| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Instead of splitting the message at spaces and checking and replacing
each word, use re.sub for the whole message.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
This is for the notifications @channel, @everyone, @group and @here.
@group isn't mentioned in the documentation[0], but it is still rendered
as a highlight in the web client, so I included it.
[0]: https://get.slack.help/hc/en-us/articles/202009646-Notify-a-channel-or-workspace
|
|
|
|
| |
Fixes #587
|
|
|
|
| |
Subteams/usergroups should never be used without the @.
|
|
|
|
|
|
|
| |
The username can't be changed by the users themselves now and is auto
generated for new users, so we shouldn't use it. Only use username as a
last resort if neither display name nor full name is set (I'm not sure
this ever happens, it looks like the full name field is mandatory).
|
|
|
|
| |
Fixes #697
|
| |
|
|
|
|
|
| |
* This change makes sure the /topic command outputs
'Topic for #random is ""' instead of
'Topic for #random is "{'value': ''}"'.
|
|
|
|
|
| |
I forgot to make some required changes in the tests after commit f3d880e
and commit 49d481c.
|
|
|
|
|
|
|
| |
12 seconds is a bit more than twice the amount of time between the pings
we send.
Fixes #238, fixes #480, fixes #561
|
|
|
|
|
|
| |
This mainly adds team, channel_general and user_alice as fixtures, so we
can use those directly instead of picking arbitrary ones from the lists.
It also adds assertions to some tests which where missing it.
|
|
|
|
| |
These tests don't do any assertions, so they are not very useful.
|
|
|
|
|
|
|
| |
Python 3 receives unicode strings as arguments in weechat callbacks,
websocet receive etc. and we can send unicode strings to weechat
functions, websocket etc., so we don't need to do any of this
converting.
|
|
|
|
|
|
| |
PluginConfig uses round-tripping through wee_slack.w to coerce
the its default values to the correct type. Without this, the normal
getters fail, as '' cannot be converted to an integer/boolean.
|
|
|
|
|
| |
This is compatible with python 3, which StringIO from StringIO or
cStringIO is not.
|
| |
|
|
|
|
| |
Comments are updated as well as live code.
|
|
|
|
|
| |
It seems pointless to serialize request metadata first with pickle, then
pack it into json, just to unpack both afterwards.
|
|
|
|
|
| |
This test doesn't do any assertions, and is as far as I can see
completely useless.
|
|
|
| |
Add @user-groups with tab-completion. @user-groups will be unfurl into format <!subteam^{ID}|handle> message before sending message via linktext method.
|
|
|
|
| |
This parameter is not used in the function.
|
|
|
|
| |
Previously, when writing e.g. `@channel: test` the colon would be lost.
|
| |
|
| |
|
|
|
|
| |
* fixes #618
|
|
|
|
| |
Simplifies the usage so we don't have to look it up each time.
|
|
|
|
|
|
|
|
| |
* Fix for #570, also incorporates #545
* Fixes from the review
* review fix #2
|
| |
|
|\
| |
| | |
Add support for shared channels
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Since shared channels only work with the new conversations API, a new
class of SlackChannel and API type were added to support them. Also
`rtm.start` doesn't include external users so their information needs to
be fetched individually when getting the channel history.
To distinguish external users from team members their nicks get appended
with an asterisk, which can be changed with a new setting. They also
appear in a new "External" group in the nicklist since their presences
can't be fetched with `users.getPresence`.
Unfortunately `conversations.history` doesn't include thread messages so
threads won't show up in shared channels.
|
| |
| |
| |
| |
| | |
These missing callbacks are not really errors or exceptions, and we
don't need to implement all of them. Raising an exception makes the
users think something isn't working, and is confusing.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* print `image_url` if it exists
* add test for image_url
* requirements.txt for pytest
* fix requirements.txt
* text before image_url
* this isn’t required
* README readability for development section
thanks @trygveaa
* README reorder for this development section
|
|
|
|
| |
Fixes: #428
|
|\
| |
| | |
feat: Remove Slacks auto prefixing of url protocols
|
| | |
|
|\ \
| | |
| | | |
Attachment tests and minor fixes
|
| | |
| | |
| | |
| | |
| | |
| | | |
The title and from_url fields of attachments doesn't seem to be escaped,
so the check if they already are in the text didn't work if the link
contained e.g. an ampersand.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This is a follow-up of commit 0345ba1. That prevented rendering the link
if it already was in the text. This prevents rendering both the
title_link and from_url if they are equal, but not in the text. In this
case it only renders the title_link.
|
| |/ |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
So we can assert on all of the return values. Since True == 1 in python,
this doesn't change the value of WEECHAT_RC_OK.
|
|/
|
|
|
|
| |
Split out the parts of topic_command_cb that's most interesting and
easiest to test into parse_topic_command and add tests for that
function.
|
|\
| |
| | |
Fix broken tests
|
| |
| |
| |
| | |
As far as I can see, this isn't used anywhere.
|
| | |
|