-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added ISbmClient for easier testing when used in other projects, adde… #18
base: master
Are you sure you want to change the base?
Conversation
…d a new constructor to allow injecting of HttpClient.
Can this be fixed and approved @prjseal ? |
I gave up on this package and made my own nuget package to call their current API. It's not hard at all. I am using Refit. |
@michaellegner I also love refit. and that's something @prjseal could invest on this repository. But unlike you, I don't give up on this repo. I know @prjseal 's competences and I follow him on Youtube. I am sure he will improve this repo. I can help if needed. |
Hi all. Sorry for not replying on here for a while. The repo isn't dead. I appreciate your comments, issues and PRs. I've recently changed jobs at work and it frees me up for more open source work. I will catch up on here and reply properly soon. |
Yes but from what I remember this repo doesn't call the right API at Slack anyway. |
I am using blocks (https://api.slack.com/block-kit) and a select number of methods in this API: https://api.slack.com/methods |
Thought maybe this makes the tests work
…d a new constructor to allow injecting of HttpClient.