Skip to content
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

Allow override of response_url on api wrapper methods #531

Closed
4 of 9 tasks
Ambro17 opened this issue Oct 6, 2019 · 2 comments
Closed
4 of 9 tasks

Allow override of response_url on api wrapper methods #531

Ambro17 opened this issue Oct 6, 2019 · 2 comments
Labels
enhancement M-T: A feature request for new functionality Version: 2x web-client
Milestone

Comments

@Ambro17
Copy link

Ambro17 commented Oct 6, 2019

Description

I want to use the pythonic api like Slack.chat_postMessage(...) when responding async to a command. As we know slack requires responses to be sent within the first 3 seconds.

If we take too long, a response_url is given to send responses there for up to 30 minutes.

Currently, the wrapper methods all fall into self.api_call base method that in its definition gets the api_url from

api_url = self._get_url(api_method)

Source code link:

What type of issue is this? (place an x in one of the [ ])

  • bug
  • enhancement (feature request)
  • question
  • documentation related
  • testing related
  • discussion

Requirements (place an x in each of the [ ])

  • I've read and understood the Contributing guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've searched for any related issues and avoided creating a duplicate issue.

  • slackclient version: 2.1.0
  • python version: 3.6
  • OS version(s): ubuntu 18
@seratch seratch added enhancement M-T: A feature request for new functionality Version: 2x web-client labels Apr 13, 2020
@seratch seratch changed the title Allow override of response url on api wrapper methods Allow override of response_url on api wrapper methods Apr 13, 2020
@seratch
Copy link
Member

seratch commented Apr 13, 2020

related to: #270

@seratch seratch added this to the 2.7.0 milestone Apr 20, 2020
@seratch
Copy link
Member

seratch commented May 20, 2020

As we'll be tracking this feature request with #270, allow me to close this issue now. Thank you very much for taking the time to share your feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement M-T: A feature request for new functionality Version: 2x web-client
Projects
None yet
Development

No branches or pull requests

2 participants