aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* every time we open a buffer, we clear the hashtableRyan Huber2016-11-221-1/+7
|
* Merge pull request #215 from terminalmage/issue190Ryan Huber2016-11-181-1/+0
|\ | | | | Remove unnecessary channels.join API request
| * Remove unnecessary channels.join API requestErik Johnson2016-04-121-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are two problems with this API request: 1. According to https://api.slack.com/methods/channels.join, "user" is not a valid argument. 2. The hash table lookup in the removed line of code is destined to fail anyway, because ``self.name`` here refers to the channel name, which is not present in the ``users`` SearchList() instance as it maps usernames to the Slack API user IDs. Indeed, this causes the call to ``users.find(self.name)`` to return ``None``, causing an AttributeError to be raised. Fixes #190.
* | Merge pull request #278 from mattrobenolt/parametrizeTollef Fog Heen2016-11-111-26/+27
|\ \ | | | | | | Leverage pytest.parametrize for test_unfurl
| * | Leverage pytest.parametrize for test_unfurlMatt Robenolt2016-11-071-26/+27
| | | | | | | | | | | | | | | This provides nicer granular test cases rather than one whole test that fails.
* | | Merge pull request #263 from rawdigits/tfheen/performanceTollef Fog Heen2016-11-111-14/+27
|\ \ \ | |/ / |/| | Performance improvements
| * | Speed up User comparisonTollef Fog Heen2016-10-031-1/+3
| | | | | | | | | | | | Avoid creating a full new string and use a string slice instead.
| * | Performance: fetch user object only onceTollef Fog Heen2016-10-031-6/+7
| | | | | | | | | | | | | | | Instead of lookup up the user object a bunch of times in buffer_prnt, just fetch it once.
| * | Performance: pass in item to updateTollef Fog Heen2016-10-031-6/+16
| | | | | | | | | | | | | | | When adding an item to the hash table, pass in the updated item, which saves a bunch of work.
| * | Slight performance optimisationTollef Fog Heen2016-10-031-1/+1
| | | | | | | | | | | | | | | | | | Looking for if something is in a list is slightly faster than counting the number of that item in a list, since if it is in the list, the loop can exit early.
* | | Fix obvious mistake with tagsTor Hveem2016-11-041-5/+5
| | | | | | | | | | | | Tag string is being overwritten instead of appended.
* | | Add support for slash commandsAlex Dills2016-10-202-1/+35
| | | | | | | | | | | | | | | Custom slash commands are handled by doing `/slack slash /foo [args…]` Fixes: #227
* | | Merge pull request #266 from irth/masterTollef Fog Heen2016-10-091-1/+6
|\ \ \ | |/ / |/| | use the correct colors for nick_prefix/nick_suffix
| * | use the correct colors for nick_prefix/nick_suffixWojciech Kwolek2016-10-091-1/+6
|/ /
* | minor tweak to support new identifiersRyan Huber2016-09-291-1/+1
| |
* | Merge pull request #262 from rawdigits/reaction_regex_improvementRyan Huber2016-09-291-1/+1
|\ \ | | | | | | add whitespace and anchors
| * | add whitespace and anchorsRyan Huber2016-09-291-1/+1
| | |
* | | Merge pull request #261 from rawdigits/mpdm_cleanupRyan Huber2016-09-291-3/+15
|\ \ \ | |/ / |/| | consistently open mpdm the same way by handling the process_
| * | do not create a new channel hereRyan Huber2016-09-291-1/+1
| | |
| * | handle mpim on websocketRyan Huber2016-09-281-0/+10
| | |
| * | consistently open mpdm the same way by handling the process_Ryan Huber2016-09-281-3/+5
|/ /
* | Merge branch 'master' of github.com:rawdigits/wee-slackRyan Huber2016-09-281-1/+8
|\ \
| * \ Merge pull request #260 from rawdigits/remove_distracting_channels_if_dont_existRyan Huber2016-09-281-1/+8
| |\ \ | | | | | | | | stop trying to hide channels that no longer exist
| | * | stop trying to hide channels that no longer existRyan Huber2016-09-281-1/+8
| | | |
* | | | put back cacheRyan Huber2016-09-281-1/+1
|/ / /
* | | tiny tweak to formattingRyan Huber2016-09-281-0/+1
| | |
* | | Merge pull request #259 from rawdigits/pep8_finallyRyan Huber2016-09-281-67/+107
|\ \ \ | |/ / |/| | pep8 errything
| * | fixed up per tfheen's suggestionsRyan Huber2016-09-281-10/+5
| | |
| * | this needs to happen. it is a mutate not an important return valueRyan Huber2016-09-281-1/+1
| | |
| * | pep8 errythingRyan Huber2016-09-281-62/+107
|/ /
* | combines mithrandir's fix and retry logic backwards fixRyan Huber2016-09-261-2/+2
| |
* | longer timeout, because big teamsRyan Huber2016-09-261-2/+2
| |
* | derpRyan Huber2016-09-261-1/+1
| |
* | Merge pull request #257 from rawdigits/fix_reconnect_logicRyan Huber2016-09-261-1/+16
|\ \ | | | | | | fixes reconnects, which hang if we never hear back from an rtm.start
| * | rushjob unfuckedRyan Huber2016-09-261-3/+3
| | |
| * | fixes reconnects, which hang if we never hear back from an rtm.startRyan Huber2016-09-261-1/+16
|/ /
* | Merge pull request #245 from rawdigits/oauth-testRyan Huber2016-09-261-0/+29
|\ \ | | | | | | add oauth code
| * | add oauth codeRyan Huber2016-06-201-0/+29
| | |
* | | Merge pull request #255 from rawdigits/tfheen/print-wrong-tokenTollef Fog Heen2016-09-251-1/+9
|\ \ \ | | | | | | | | Print out first ten characters of erroring tokens
| * | | Print out first ten characters of erroring tokensTollef Fog Heen2016-09-251-1/+9
|/ / / | | | | | | | | | | | | | | | | | | | | | If you have multiple tokens, it can be hard to figure out which one is causing an error, so print out the first ten characters to help identify the problematic one. Fixes: #163
* | | Do minimal HTML entity decodingTollef Fog Heen2016-09-251-0/+3
| | | | | | | | | | | | | | | | | | Just handle <>& for now. Fixes: #186
* | | Handle @@ and ## and similar correctlyTollef Fog Heen2016-09-241-10/+11
| | | | | | | | | | | | | | | | | | | | | Make sure we have something that's a well-formed nick before we go ahead and do silly things. Fixes: #237
* | | Clarify where main buffer messages come fromTollef Fog Heen2016-09-241-1/+1
| | | | | | | | | | | | | | | | | | | | | Make it clearer where the messages are coming from when printing to main buffer. Fixes: #240
* | | Continue even on cache loading errors.Tollef Fog Heen2016-09-241-0/+3
| | | | | | | | | | | | | | | | | | | | | If we get a ValueError when loading the json, continue, instead of making the user unhappy. Closes: #248
* | | Merge pull request #225 from megalithic/feature/weechat-highlightsTollef Fog Heen2016-09-241-3/+3
|\ \ \ | | | | | | | | Support passing additional tags to enable notification_center.py
| * | | Initial add to trigger weechat highlights/notificationsSeth Messer2016-05-061-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | More updates to get highlights working add that gitignore Hopefully handles all situations where a message would have highlights Removed additional settings; was not needed Resolve code review items - fix an erroneously removed condition for unsetting a config option - remove notify_highlight tag; is redundant - thanks @tfheen! Removes project-local .gitignore Undo an uncomment Reverting back to including the previously commented line
* | | | Print out topic when the user does /topicTollef Fog Heen2016-09-241-2/+8
| | | |
* | | | Ignore IRC commands for non-slack buffersTollef Fog Heen2016-09-241-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we're in a non-slack buffer, just ignore /part, /topic, /me, etc. This is somewhat based off #212, so thanks to @bendem for pointing this out. Fixes: #137
* | | | Update topic in channel object when changing the topicTollef Fog Heen2016-09-241-2/+2
| | | |
* | | | Wait 30 seconds not 3, because we get rate limited and never reconnect.Ryan Huber2016-09-221-1/+1
| | | |
generated by cgit (git 2.34.1) at 2024-11-16 02:16:08 +0000