| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Having it in slack/main.py doesn't work with combining to one file.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
The formatting can change between major versions of black, so make sure
we're always using the same one.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
On some Linux systems org.freedesktop.Secret D-Bus service is not
available (installed/enabled), typically as a deliberate choice. In
those cases `extract_token_from_browser.py` hands for the duration of
D-Bus timeout and eventually fails. Note that it does not handle the
timeout exception, so a fallback is not used.
This commit adds `--no-secretstorage` option which skips using that
service.
|
|
|
|
|
|
|
| |
Previously running /thread with a message id for a thread reply (or
pressing T on the message in cursor mode, which is the same thing) would
open a buffer with just the thread reply. Now the thread buffer for the
thread the message is in is opened instead.
|
|
|
|
|
|
|
|
|
| |
If a reaction has more than 50 users, the list of users will be cropped
to just 50 users. This made the counts wrong since we used the number of
users rather than the count property. Fix it by using the count property
instead and adding "and others" to the list of users if
show_reaction_nicks is enabled (the same as the Slack client shows when
you hover over the reaction).
|
|
|
|
| |
Fixes #912
|
| |
|
| |
|
|
|
|
|
|
|
| |
The caching doesn't seem to help much and it spends a while in the post
step for the cache so the total runtime is lower without the cache.
Having linting and testing in separate jobs doesn't seem valuable to me
either.
|
|
|
|
|
|
|
|
| |
I tried to do a fix for this in commit 6478c141, but it only fixed
normal text, not e.g. code blocks and attachments. With this I think it
should be correct everywhere.
Fixes #908
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The extract_token_from_browser.py script does not handle Firefox containers
correctly. The container support works by namespacing cookies and local storage
with a 'userContextId'. In effect this means that when containers are used,
running the script will select the cookie from whichever container happens to be
returned first by the database query, which will be combined with the
non-containerised instance of the local storage.
Add correct handling by adding a new --container flag which specifies a
container either by numeric ID or by name, and including the correct
userContextId into the cookie query and local storage path (or excluding all
cookies from containers from the query if no --container flag is supplied).
With this the script can be run like:
python extract_token_from_browser.py --container Work firefox
to get the Slack auth token for the 'Work' container.
Signed-off-by: Toke Høiland-Jørgensen <toke@toke.dk>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Slack documentation claims that the “ts” flag on attachments
is “An integer Unix timestamp”. However, evidently it can be in
milliseconds (i.e., a timestamp multiplied by 1000), causing
out-of-range exceptions when trying to feed them to date.fromtimestamp().
This can cause all messages after such attachments to just vanish,
as if they were never there, as parsing stops.
Accept both; as a heuristic, the Slack web interface seems to use
100000000000 as a threshold, so we do the same and divide by 1000
if so.
Co-authored-by: Steinar H. Gunderson <steinar+git@gunderson.no>
|
|
|
|
|
| |
It already marks it as read when `/input set_unread_current_buffer` is
run, but this command was replaced with `/buffer set unread` in WeeChat 3.8.
|
|
|
|
|
|
|
|
|
| |
Only subscribed threads can be marked as read. This means that if we
open other threads, they will be opened every time you reload wee-slack
which is impractical and annoying. Previously, any thread with a
highlight (in WeeChat, not necessarily a Slack highlight) was opened.
Fixes #803, fixes #830
|
|
|
|
|
|
| |
This fixes a regression in commit 74da303 which caused certain messages
to include the bold/italic/strikethrough characters (but not the color
attributes) even though the style should not be active.
|
| |
|