search

lamby / django-slack

Slack integration for Django, using the templating engine to generate messages
https://django-slack.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
241 stars 57 forks source link

Unable to get the response of Slack API #82

Closed hexchain closed 6 years ago

hexchain commented 6 years ago

Currently, django-slack does not provide the response of the Slack API, making it impossible to update a sent message afterwards (which requires the ts value from the response).

Is this specially designed so, or just a missing feature?

hexchain commented 6 years ago

It seems that it is not easy to provide the return value consistently - some backends are dummy ones that do not return, some are asynchronous that cannot provide API response immediately.

lamby commented 6 years ago

@hexchain wrote:

some are asynchronous that cannot provide API response immediately.

Indeed. However, the synchronous ones should and could definitely use this. Can you provide a patch? :)

hexchain commented 6 years ago

The response can still be provided for the Celery backend if result storage is configured for celery, but that would introduce another return type for send_message.

lamby commented 6 years ago

Released as 5.12.0 — thank you.