aboutsummaryrefslogtreecommitdiffstats
path: root/wee_slack.py
Commit message (Collapse)AuthorAgeFilesLines
* fix: corrected registration instructionsDaniel Brendle2018-05-141-1/+1
| | | | | | | | in the registration dialog, wee_slack asks the user to reload the slack script via /script. in my weechat 2.1 on debian sid, this is not the proper way to reload scripts anymore. instead one has to use the /python command to achieve the same effect. this commit corrects the misleading instruction message.
* Make sure to compare nick against uncoloured nick when choosing tagsTollef Fog Heen2018-05-061-1/+1
| | | | Fixes #531
* Merge pull request #346 from V13Axel/status_as_dm_topicTrygve Aaberge2018-04-281-6/+35
|\ | | | | Add emojis to status as a DM topic
| * Include real name in dm channel topicTrygve Aaberge2018-04-131-1/+5
| |
| * Don't show 'None' when status emoji or text is missingTrygve Aaberge2018-04-131-2/+2
| |
| * Fix finding dm channel after merge with masterTrygve Aaberge2018-04-131-2/+3
| | | | | | | | Finding by name didn't work anymore.
| * Refactor finding a channel by members into a methodTrygve Aaberge2018-04-131-6/+7
| |
| * Sync branch 'status_as_dm_topic' with masterTrygve Aaberge2018-04-131-0/+23
| |\
| | * Move formatting to its own functionV13Axel2018-04-121-2/+7
| | |
| | * Add in emoji to status as dm topicV13Axel2018-04-121-4/+5
| | |
| | * topic is user's statusRyan Huber2017-04-131-0/+14
| | |
* | | Fix nick sanitizationAlex Crawford2018-04-271-2/+1
|/ / | | | | | | This now properly replaces spaces with underscores.
* | Change slack away/back to use users.setPresence (#559)Paulo Cesar Zanco2018-04-121-2/+2
| | | | | | Fixes #552
* | Fix usage check in slash commandTrygve Aaberge2018-04-111-2/+2
| | | | | | | | | | | | The args argument will be 'slash' when not giving any arguments to the slash command, it will never be None. The `server` variable doesn't exist anymore, use `w.prnt` instead.
* | Remove incorrect validation check for status commandTrygve Aaberge2018-04-111-4/+0
| | | | | | | | | | | | The args argument is a string and includes 'status', so it will never be None. Additionally, the status command works without any arguments (which cleares the status), so it shouldn't have such a check.
* | Fix /msg user messageTollef Fog Heen2018-03-141-1/+1
| | | | | | | | | | | | | | `command_talk` wants a string it can pick apart. Pass that in from `msg_command_cb`. Fixes: #530
* | Fix self-nick color in messagesJoan Touzet2018-03-141-4/+3
| |
* | Avoid logging the entire slack token, just do the prefixTollef Fog Heen2018-03-131-2/+2
| |
* | Use StringIO instead of string concatenationTollef Fog Heen2018-03-111-10/+10
| | | | | | | | | | | | | | This makes a big difference for larger teams where rtm.start can time out just due to Python's strings being immutable. Fixes: #516
* | Re-add support for weechat.look.nick_{prefix,suffix}Tollef Fog Heen2018-03-111-2/+14
| | | | | | | | Fixes: #428
* | 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
| |
* | 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).
* | 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`)
* | | 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.
* | | 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-281-0/+51
|\ \ \ \ | |/ / / |/| | | Restore OAuth support
| * | | Respond to Trygve's commentsA. Jesse Jiryu Davis2018-01-271-11/+11
| | | |
| * | | 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
| | | |
* | | | Fix presence subscription now mandatory to get state changesIsmaël Bouya2018-01-271-3/+20
| | | | | | | | | | | | | | | | https://api.slack.com/docs/presence-and-status#batching
* | | | 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.
* | | | 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-161-11/+29
|\ \ \ | |/ / |/| | 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.