Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Check that the bot_id exists before using it | Trygve Aaberge | 2017-08-17 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | Instead of just checking that bot_id is not None, check that it exists in the list of bots. This prevents a KeyError when trying to access a bot that doesn't exist in the list. This KeyError would happen if you use a reminder. The message from slackbot then has bot_id B01 which is not in the list of bots from rtm.start and gives bot_not_found if trying to look it up with the bots.info API endpoint. By ignoring the bot_id we fall back to the user/username, which in the case of reminders is slackbot, so the name appears just as the other messages from slackbot. Fixes #420 | ||||
* | Merge pull request #417 from trygveaa/fix-html-entity-encoding | Tollef Fog Heen | 2017-08-01 | 2 | -4/+13 |
|\ | | | | | Fix HTML entity-encoding of outgoing messages | ||||
| * | Fix HTML entity-encoding of outgoing messages | Trygve Aaberge | 2017-08-01 | 2 | -4/+13 |
|/ | | | | | | We have to replace the & before we replace < and >, otherwise the & in < and > are going to be replaced. Additionally, we need to end the sequences with ; which was missing. | ||||
* | Merge remote-tracking branch 'origin/nicklist-presence-groups' | Tollef Fog Heen | 2017-07-28 | 1 | -1/+12 |
|\ | |||||
| * | Merge pull request #403 from marceldev89/nicklist-presence-groups | Tollef Fog Heen | 2017-07-28 | 1 | -1/+12 |
| |\ | | | | | | | Update nicklist when team member presence changes | ||||
| | * | Move presence change to a SlackTeam method | Marcel de Vries | 2017-07-24 | 1 | -6/+9 |
| | | | |||||
| | * | Update nicklist when team member presence changes | Marcel de Vries | 2017-07-23 | 1 | -1/+9 |
| |/ | |||||
* | | Remove /slack join command from README.md | Tollef Fog Heen | 2017-07-28 | 1 | -1/+0 |
| | | | | | | | | Fixes: #362 | ||||
* | | Decode the basic HTML entities in topic handling | Tollef Fog Heen | 2017-07-28 | 1 | -6/+9 |
| | | | | | | | | Closes: #315 | ||||
* | | Merge pull request #399 from wee-slack/nicklist-presence-groups | Tollef Fog Heen | 2017-07-28 | 1 | -11/+15 |
|\| | | | | | Put users in groups according to presence | ||||
| * | Put users in groups according to presence | Tollef Fog Heen | 2017-07-19 | 1 | -11/+15 |
| | | | | | | | | Fixes #398 | ||||
* | | Merge pull request #408 from auscompgeek/fix-status-cmd | Tollef Fog Heen | 2017-07-28 | 1 | -1/+1 |
|\ \ | | | | | | | Fix AttributeError on /slack status in team buffer | ||||
| * | | Fix AttributeError on /slack status in team buffer | David Vo | 2017-07-27 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #412 from trygveaa/set-correct-unread-count | Tollef Fog Heen | 2017-07-28 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Set correct number of unread messages in hotlist | ||||
| * | | | Set correct number of unread messages in hotlist | Trygve Aaberge | 2017-07-28 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | While the commit 6718e2f fixed buffers not appearing in the hotlist, after it the number of new messages for each buffer was gone. This makes the number of messages for each buffer shown again. | ||||
* | | | | Merge pull request #411 from auscompgeek/fix-open-thread | Tollef Fog Heen | 2017-07-28 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | Fix opening thread buffers | ||||
| * | | | | Fix opening thread buffers | David Vo | 2017-07-28 | 1 | -1/+1 |
| |/ / / | |||||
* | | | | Add support for editing other messages than just the last one | Tollef Fog Heen | 2017-07-28 | 2 | -6/+15 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Similar syntax as for adding reactions: 3s/foo/bar/ Fixes: #153 | ||||
* | | | | Fix return codes in buffer_input_callback | Tollef Fog Heen | 2017-07-28 | 1 | -2/+2 |
|/ / / | | | | | | | | | | Not sure this actually matters as weechat seemed happy with the wrong return codes too | ||||
* | | | Merge pull request #406 from ToxicFrog/toxicfrog/escape | Ben Kelly | 2017-07-26 | 1 | -0/+7 |
|\ \ \ | |/ / |/| | | Properly escape <&> when sending messages to Slack. | ||||
| * | | Properly escape <&> when sending messages to Slack. | Ben Kelly | 2017-07-25 | 1 | -0/+7 |
|/ / | | | | | | | | | Signed-off-by: Ben Kelly <bk@ancilla.ca> Signed-off-by: Ben Kelly <btk@google.com> | ||||
* | | Merge pull request #394 from ToxicFrog/toxicfrog/typing-notifications | Ben Kelly | 2017-07-21 | 1 | -1/+1 |
|\ \ | | | | | | | Don't send typing notifications for threads, since Slack doesn't like that | ||||
| * | | Don't send typing notifications for threads, since Slack doesn't like that. | Ben Kelly | 2017-07-13 | 1 | -1/+1 |
| | | | | | | | | | | | | Signed-off-by: Ben Kelly <btk@google.com> | ||||
* | | | Re-add setting of localvar_set_nick | Tollef Fog Heen | 2017-07-21 | 1 | -0/+3 |
| | | | | | | | | | | | | Fixes #80 (again) | ||||
* | | | Handle /msg * foo correctly | Tollef Fog Heen | 2017-07-21 | 1 | -1/+4 |
| | | | | | | | | | | | | | | | | | | Send * to the current channel. Fixes: #191 | ||||
* | | | Fix use-before-assignment when fetching channel to send to | Tollef Fog Heen | 2017-07-21 | 1 | -1/+1 |
| |/ |/| | |||||
* | | Merge pull request #397 from gilbertw1/feature-set-channel-unread-count | Tollef Fog Heen | 2017-07-19 | 1 | -19/+41 |
|\ \ | |/ |/| | | | | | Use channel.open and im.open to get unread counts. Fixes #396 | ||||
| * | restore call to update_nicklist() (accidental removal) | Bryan Gilbert | 2017-07-19 | 1 | -0/+1 |
| | | |||||
| * | use im.open endpoint to properly update unread count for ims on startup | Bryan Gilbert | 2017-07-19 | 1 | -6/+17 |
| | | |||||
| * | add group info request and handler to set unread count and update hotlist | Bryan Gilbert | 2017-07-17 | 1 | -0/+9 |
| | | |||||
| * | call channel info on channel open to set unread count and update hotlist | Bryan Gilbert | 2017-07-16 | 1 | -19/+20 |
|/ | |||||
* | Merge pull request #371 from ToxicFrog/toxicfrog/mrkdwn | Ben Kelly | 2017-07-13 | 1 | -8/+13 |
|\ | | | | | Disable formatting for messages with `mrkdwn` unset | ||||
| * | Disable formatting for messages with `mrkdwn` unset | Ben Kelly | 2017-07-12 | 1 | -8/+13 |
| | | | | | | | | | | Signed-off-by: Ben Kelly <btk@google.com> Signed-off-by: Ben Kelly <bk@ancilla.ca> | ||||
* | | Merge pull request #392 from ToxicFrog/toxicfrog/fix-tests | Ben Kelly | 2017-07-13 | 3 | -2/+11 |
|\ \ | | | | | | | Fix the tests | ||||
| * | | Fix the tests | Ben Kelly | 2017-07-12 | 3 | -2/+11 |
|/ / | | | | | | | | | | | | | | | | | | | Not everything that needed to be mocked out was, and there was also an issue with every instance of PluginConfig sharing the same mutable settings field and thus stomping on each other if you initialized more than one in the same test. Signed-off-by: Ben Kelly <bk@ancilla.ca> Signed-off-by: Ben Kelly <btk@google.com> | ||||
* | | Merge pull request #372 from ToxicFrog/toxicfrog/setting-desc | Ben Kelly | 2017-07-12 | 1 | -26/+97 |
|\ \ | |/ |/| | Add descriptions for weeslack plugin settings. | ||||
| * | Add descriptions for weeslack plugin settings. | Ben Kelly | 2017-07-12 | 1 | -26/+97 |
|/ | | | | | Signed-off-by: Ben Kelly <btk@google.com> Signed-off-by: Ben Kelly <bk@ancilla.ca> | ||||
* | Merge pull request #350 from ToxicFrog/toxicfrog/italics | Ben Kelly | 2017-07-12 | 1 | -12/+27 |
|\ | | | | | Add italic support, and settings to switch between italic and underline | ||||
| * | Add italic support, and settings to switch between italic and underline | Ben Kelly | 2017-04-15 | 1 | -12/+27 |
| | | | | | | | | | | Signed-off-by: Ben Kelly <btk@google.com> Signed-off-by: Ben Kelly <bk@ancilla.ca> | ||||
* | | Merge pull request #391 from trygveaa/fix/nick-coloring | Ben Kelly | 2017-07-12 | 2 | -14/+11 |
|\ \ | | | | | | | Fix nick coloring in weechat < 1.5 | ||||
| * | | Remove documentation for colorize_nicks | Trygve Aaberge | 2017-07-12 | 1 | -10/+0 |
| | | | | | | | | | | | | | | | | | | This option was removed in 925278f, in favor of using weechat's color handling. I.e. you now set this with weechat's core options instead of a specific option for this script. | ||||
| * | | Get nick color name correctly in weechat < 1.5 | Trygve Aaberge | 2017-07-12 | 1 | -2/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In weechat 1.5, the info name 'irc_nick_color_name' was renamed to 'nick_color_name'. Since we want to support weechat < 1.5 and don't want deprecation warnings in >= 1.5, we need to check the version and use the appropriate name. Fixes #364 | ||||
| * | | Rename version variable to weechat_version | Trygve Aaberge | 2017-07-12 | 1 | -2/+2 |
|/ / | | | | | | | | | This is going to be used another place in the script in the next commit, so set a more verbose name so it's not confused with the script version. | ||||
* | | Merge pull request #384 from cmatheson/fix-349 | Ryan Huber | 2017-07-11 | 1 | -1/+2 |
|\ \ | | | | | | | Stop polling dead websockets. | ||||
| * | | Stop polling dead websockets. Fixes #349 | Cameron Matheson | 2017-07-05 | 1 | -1/+2 |
|/ / | |||||
* | | Correct URL in README | Tollef Fog Heen | 2017-05-02 | 1 | -2/+2 |
| | | |||||
* | | Merge pull request #351 from wee-slack/settings_cleanup | Ryan Huber | 2017-04-21 | 1 | -10/+0 |
|\ \ | | | | | | | Clean up a little bit | ||||
| * | | Remove colorize_messages setting | Tollef Fog Heen | 2017-04-16 | 1 | -1/+0 |
| | | | | | | | | | | | | This doesn't seem to be used, so just remove it. | ||||
| * | | Remove dead code related to cache | Tollef Fog Heen | 2017-04-16 | 1 | -9/+0 |
| | | | | | | | | | | | | The cache is gone, remove some related code. | ||||
* | | | Merge pull request #356 from ToxicFrog/toxicfrog/missing-text | Ryan Huber | 2017-04-21 | 1 | -2/+2 |
|\ \ \ | | | | | | | | | Fix channel history not appearing when an earlier message has 'text': None |