search

os / slacker

Full-featured Python interface for the Slack API
Apache License 2.0
1.6k stars 245 forks source link

Blocks #152

Closed genghis closed 4 years ago

genghis commented 5 years ago

Just want to be able to use blocks with this.

SwiftyNZ commented 5 years ago

This would be helpful.

mrshwah commented 5 years ago

I could definitely use this in a current project, would love to see it added.

genghis commented 5 years ago

I could definitely use this in a current project, would love to see it added.

You can either pull my changes (https://github.com/genghis/slacker) until it's pulled into Slacker proper, or you can just make the alterations yourself in the post_message class (literally just add blocks as a param) and all the relevant functions within it.

mrshwah commented 5 years ago

@genghis Yep, I've been using your version (works great), just wanted to add my support for the pull!

symroe commented 5 years ago

Thank you, @genghis!

This pull request works well in some testing.

Can I suggest to @genghis that you squash the commits in to one, and give it a meaningful title and description?

Given that the Slack documentation is pushing users of the API towards blocks now, this PR is important for making the library useful at all.

As it's a small change, I would like to add my support to the PR being merged, and a new version cut :)

botzill commented 5 years ago

Hi

Any updates on this PR ?

symroe commented 5 years ago

I have no association with the project, but the only update I have is that I made a PR with the same changes but a slightly cleaner commit history, here #156

I'm using this, but have no feedback on if it's a useful contribution or not.

senpay commented 4 years ago

@genghis @botzill Hello! I am making a fork now and will publish an updated version on pypi under some other name. Please feel free to suggest your pr here: https://github.com/senpay/slacker