slackapi / slackapi/python-slack-sdk
Improve structure of `admin.apps.config.set` HTTP API arguments
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 4k
- Forks
- 857
- Avg merge
- 22h 21m
- Merged PRs (30d)
- 16
Description
I improved the documentation and helped implement the API methods for admin.apps.config.* recently in the node-slack-sdk. As part of this work, it became clear that the arguments for the admin.apps.config.set method were vaguely documented and unclear.
I did some discovery and was able to more specifically type them in the node-slack-sdk. See this section of the PR in node-slack-sdk.
Perhaps we could do something similar in the Python SDK.
Category (place an x in each of the [ ])
- slack_sdk.web.WebClient (sync/async) (Web API client)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the Python SDK entry point for the admin.apps.config.set Web API method and compare its argument handling with the referenced node-slack-sdk PR, especially the linked diff. Define the argument structure consistently with that reference and verify that the method's accepted shapes are clearly represented in the SDK.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100