Skip to content

Conversation

@Roach
Copy link
Contributor

@Roach Roach commented Dec 15, 2016

PR Summary

Release 1.0.3 broke the ability to search for user by id. This restores that functionality.

Related Issues

Fixes #151

Test strategy

Added tests for searching for user by id

✅ I've read and understood the Contributing guidelines
✅ I've read and agree to the Code of Conduct
✅ I've been mindful about doing atomic commits, adding documentation to my changes, not refactoring too much.
✅ I've a descriptive title and added any useful information for the reviewer.
✅ I've written tests to cover the new code and functionality included in this PR.
✅ I've read, agree to, and signed the Contributor License Agreement

@Roach Roach requested a review from UncannyBingo December 15, 2016 22:57
@Roach Roach added bug labels Dec 15, 2016
@codecov-io
Copy link

codecov-io commented Dec 15, 2016

Current coverage is 62.59% (diff: 100%)

Merging #152 into master will increase coverage by 0.72%

@@             master       #152   diff @@
==========================================
  Files             8          8          
  Lines           257        262     +5   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            159        164     +5   
  Misses           98         98          
  Partials          0          0          

Powered by Codecov. Last update 05e4bd8...d8fcffd

Copy link

@UncannyBingo UncannyBingo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@UncannyBingo UncannyBingo merged commit d181101 into master Dec 15, 2016
@UncannyBingo UncannyBingo deleted the roach/fix-find-user-by-id branch December 15, 2016 23:10
@TheKevJames
Copy link

TheKevJames commented Dec 15, 2016

@Roach or @DEGoodmanWilson I see you released this as 1.0.4 -- thanks for getting that out so quickly! Just wanted to point out in case it was an oversight that you didn't push the setup.py change for releasing as 1.0.4. I got a bit confused to see a new release come out without a code change!

@Roach
Copy link
Contributor Author

Roach commented Dec 16, 2016

@TheKevJames d'oh! fixed that, too: #153

c-goosen pushed a commit to c-goosen/python-slackclient that referenced this pull request Jun 18, 2019
* Fixed the ability to search for a user by ID

* Added test for invalid user search
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants