SlackBotMessages icon indicating copy to clipboard operation
SlackBotMessages copied to clipboard

Added ISbmClient for easier testing when used in other projects, adde…

Open ianbeemsterboerr opened this issue 5 years ago • 6 comments

…d a new constructor to allow injecting of HttpClient.

ianbeemsterboerr avatar Sep 23 '20 15:09 ianbeemsterboerr

Can this be fixed and approved @prjseal ?

AntonioKoB avatar Mar 05 '22 13:03 AntonioKoB

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 avatar Mar 05 '22 14:03 michaellegner

@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.

AntonioKoB avatar Mar 05 '22 20:03 AntonioKoB

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.

prjseal avatar Mar 05 '22 20:03 prjseal

@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.

Yes but from what I remember this repo doesn't call the right API at Slack anyway.

michaellegner avatar Mar 05 '22 20:03 michaellegner

I am using blocks (https://api.slack.com/block-kit) and a select number of methods in this API: https://api.slack.com/methods

michaellegner avatar Mar 05 '22 20:03 michaellegner