aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Avoid erroring for reactions on filesTollef Fog Heen2018-03-101-2/+2
| | | | | | | | If somebody adds a reaction on a file, there will be no channel associated with it. Handle that gracefully. (Reaction will still be missing, but at least there's no traceback.) Related to #316.
* Implement /whois commandTollef Fog Heen2018-03-091-0/+29
| | | | Fixes: 507, based on the work done in that PR by @Shaac.
* Make string formatting of Slack Teams be more usefulTollef Fog Heen2018-03-091-0/+3
|
* Drop is_bot and format bot messages as such againTollef Fog Heen2018-03-091-6/+8
| | | | | | | | is_bot seems to no longer be a thing, so drop that setting. For messages from bots with username set in the message, use that username, but mark it as a bot message. Fixes #519
* Reorder get_sender bots vs user checksLuK13372018-03-091-4/+4
| | | | | | * Fixes display names for 'app' users, in before it'd be displayed as "bot :]" and now it's "-{display name}-".
* Release v2.0.0v2.0.0Trygve Aaberge2018-03-031-1/+1
|
* Remove outdated info from readmeTrygve Aaberge2018-03-031-12/+6
| | | | The refactor is a long time ago now, and caching has been removed.
* Merge pull request #501 from trygveaa/dont-tag-team-messages-as-normal-messagesTrygve Aaberge2018-03-031-2/+2
|\ | | | | Don't tag team messages as normal messages
| * Add slack_{type} to the message tag listTrygve Aaberge2018-02-181-1/+1
| | | | | | | | | | Makes it easier to people to filter slack messages the way they want (e.g. if you don't want team messages in the hotlist).
| * Prevent highlights on team messagesTrygve Aaberge2018-02-181-1/+1
| | | | | | | | | | | | | | | | | | | | If the user has set up highlight for e.g. their own nick, the team buffer will trigger a highlight when wee-slack is loaded because it prints the nick. You will also get a highlight from e.g. /slack users. The messages we send to the team buffer are informational messages that I don't think should trigger highlights. They will still show up in the hotlist as normal messages.
| * Don't tag team messages as private messagesTrygve Aaberge2018-02-181-1/+1
|/ | | | | Team messages shouldn't show up in the hotlist as private messages (default green), but as normal messages (default white).
* debugging helpCharlie Allom2018-02-161-0/+9
| | | | | In #504 I proposed to add some help to future debuggers. This is a split PR from that. cc: @trygveaa
* Merge pull request #502 from trygveaa/nicklist-dont-add-deleted-usersTrygve Aaberge2018-02-051-0/+2
|\ | | | | Don't add deleted users to the nicklist
| * Don't add deleted users to the nicklistTrygve Aaberge2018-02-031-0/+2
|/ | | | | | | | | | | | | This check was done when generating complete nicklist, but not when updating a single user. Deleted users are still part of the member list of a channel, if they don't leave the channel first. This caused a deleted user to be inserted into the nicklist when processing a joined channel message or when receiving a presence change for that user. When investigating this, I saw that we still receive presence change events for deleted users if we subscribe to them. We might want to filter out the deleted users when generating the list of users to subscribe to.
* Merge pull request #490 from SandyRogers/autocomplete-nicks-in-threadsTrygve Aaberge2018-02-021-4/+6
|\ | | | | Include members set and team in SlackThreadChannel
| * Use self.team throughtout SlackThreadChannel classSandy Rogers2018-02-021-5/+5
| |
| * Include members set and team in SlackThreadChannelSandy Rogers2018-01-171-0/+2
| | | | | | Allows autocompletion of nicks to work (instead of error from `nick_completion_cb`)
* | Merge pull request #500 from trygveaa/fix-quit-callbackTrygve Aaberge2018-01-311-2/+2
|\ \ | | | | | | Fix quit callback function
| * | Fix quit callback functionTrygve Aaberge2018-01-301-2/+2
|/ / | | | | | | | | The function was renamed at some point without renaming the hook signal, so weechat would print an error when exiting.
* | Merge pull request #499 from immae/presence_subscriptionTrygve Aaberge2018-01-291-1/+6
|\ \ | | | | | | Temporary fix user presence subscription crashing big teams
| * | Temporary fix user presence subscription crashing big teamsIsmaël Bouya2018-01-291-1/+6
|/ /
* | Merge pull request #494 from ToxicFrog/toxicfrog/display-namesBen Kelly2018-01-281-7/+14
|\ \ | | | | | | SlackUser: use profile.display_name if available
| * | SlackUser: use profile.display_name if availableBen Kelly2018-01-281-7/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since recent changes in Slack, usernames can no longer be set directly, and are instead inferred from the email address. This is not particularly useful, so the recommendation is to instead use display_name. display_name may have spaces in it, however, which confuses tab completion. With this change, wee-slack uses display name for both display and completion, after first stripping all whitespace from it. Ref linking (via linkify_text) ensures that @mentions continue to work properly even when this whitespace removal means that the name being @mentioned locally doesn't match the "real" username.
* | | Merge pull request #487 from ajdavis/restore-slack-oauthTrygve Aaberge2018-01-282-1/+77
|\ \ \ | |/ / |/| | Restore OAuth support
| * | Respond to Trygve's commentsA. Jesse Jiryu Davis2018-01-272-14/+14
| | |
| * | update README for new /slack register commandA. Jesse Jiryu Davis2018-01-171-1/+26
| | |
| * | oauth cleanupsA. Jesse Jiryu Davis2018-01-171-36/+39
| | |
| * | add token once oauth completesA. Jesse Jiryu Davis2018-01-171-2/+16
| | |
| * | update oauth logic for new command codeA. Jesse Jiryu Davis2018-01-151-5/+8
| | |
| * | add oauth codeRyan Huber2018-01-151-0/+31
| | |
* | | Merge pull request #496 from immae/presence_subscriptionTrygve Aaberge2018-01-271-3/+20
|\ \ \ | | | | | | | | Fix presence subscription now mandatory to get state changes
| * | | Fix presence subscription now mandatory to get state changesIsmaël Bouya2018-01-271-3/+20
|/ / / | | | | | | | | | https://api.slack.com/docs/presence-and-status#batching
* | | Merge pull request #495 from ToxicFrog/toxicfrog/sslwantreadBen Kelly2018-01-201-0/+3
|\ \ \ | | | | | | | | Suppress SSLWantReadError errors
| * | | Suppress SSLWantReadError errorsBen Kelly2018-01-201-0/+3
|/ / / | | | | | | | | | | | | This exception is expected to some degree and does not affect the functioning of wee-slack. With this change, it will no longer appear in the log.
* | | Merge pull request #493 from ToxicFrog/toxicfrog/nicklistBen Kelly2018-01-201-5/+8
|\ \ \ | |_|/ |/| | Handle memberlist updates in channelsinfo message
| * | Handle memberlist updates in channelsinfo messageBen Kelly2018-01-191-5/+8
|/ / | | | | | | | | | | | | In large Slacks, channel membership isn't included in the initial channel info, but in channelsinfo messages sent after connection. This fixes an issue where most users would not ever show up in the nicklist when connecting to such slacks.
* | Merge pull request #437 from trygveaa/completion-of-custom-emojisTrygve Aaberge2018-01-162-11/+37
|\ \ | |/ |/| Add completion of custom emojis
| * Reload emoji list on emoji change eventTrygve Aaberge2018-01-141-0/+5
| | | | | | | | | | | | | | | | | | | | It's simpler to just reload the whole list instead of adding and removing specific emojis, especially since we would have to remove aliases for an emoji when the emoji is removed (and we don't store if an emoji is an alias or not). Since this probably doesn't happen that often, it shouldn't be a problem to do it this way.
| * Ensure that EMOJI is always definedTrygve Aaberge2017-09-171-7/+7
| | | | | | | | | | | | | | After the last commit, the EMOJI variable is accessed in the initialization of SlackTeam. This made some tests fail since they don't run the main code, so it would not be defined. By setting EMOJI to an empty list in the top, we ensure that this is not a problem.
| * Add completion of custom emojisTrygve Aaberge2017-09-171-2/+19
| | | | | | | | | | | | | | | | | | | | This loads the list of custom emojis for each team after wee-slack has connected to the team, and adds them to the emoji completion list for that team. The list of custom emojis is only loaded if the list of the standard emojis has been loaded, since it would be a bit strange to only have completions of the custom emojis.
| * Add documentation for emoji completionsTrygve Aaberge2017-09-171-0/+8
| |
| * Refactor emoji list loadingTrygve Aaberge2017-09-171-7/+5
| | | | | | | | | | This prevents an error that EMOJI is not defined when trying to tab complete after the emoji list could not be loaded.
| * Improve error message when weemoji.json is not foundTrygve Aaberge2017-09-171-2/+2
| | | | | | | | | | | | | | | | By printing the exception instead of sys.exc_info, we get more useful error messages, including the path to the file it is trying to load if it can't find it. Fixes #431
| * Remove unnecessary dbg statementTrygve Aaberge2017-09-171-2/+0
| | | | | | | | | | Emoji completion works fine for me without it, so this doesn't seem to be necessary anymore.
* | Merge pull request #444 from trygveaa/feat/unfurl-prevent-auto-linkingTrygve Aaberge2018-01-142-7/+54
|\ \ | | | | | | feat: Remove Slacks auto prefixing of url protocols
| * | test: Add tests for unfurl_auto_link_displayTrygve Aaberge2017-10-011-1/+24
| | |
| * | refactor: Get config variables in unfurl_refsTrygve Aaberge2017-10-011-7/+14
| | | | | | | | | | | | | | | Instead of each of the callers of unfurl_refs having to get the config values, get them in unfurl_refs if they are not set by the caller.
| * | feat: Make it possible to only display url when text matchesTrygve Aaberge2017-10-011-9/+14
| | | | | | | | | | | | | | | | | | This changes the feature from the previous commit so it's also possible to only display the url when the text and the url (without the protocol) in a link is equal.
| * | feat: Remove Slacks auto prefixing of url protocolsTrygve Aaberge2017-10-011-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When you write multiple words separated by dots, or email adresses, Slack will turn those into links. Wee-slack will display this as `text (url)`, where url is just the text with http:// or mailto: prefixed. This makes the text hard to read, and might not even make sense (e.g. when writing server names or user@hostname). When you receive such an url, the alt text of the url will be the original text. This means that we can check if the url is just the alt text with a protocol prefixed. If it is, we should just print the alt text and ignore the url. I guess some people may want the urls, so they are able to click the links. And since it removes information coming from slack, I made it into an option instead of making it the default.
* | | Merge pull request #440 from trygveaa/use-last-msg-timestamp-for-mark-readTrygve Aaberge2018-01-141-23/+22
|\ \ \ | | | | | | | | Use the timestamp of the last message in mark_read
generated by cgit (git 2.34.1) at 2024-11-13 04:05:49 +0000