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

Improve structure of admin.apps.config.set HTTP API arguments #1412

Open
1 task done
filmaj opened this issue Oct 6, 2023 · 1 comment
Open
1 task done

Improve structure of admin.apps.config.set HTTP API arguments #1412

filmaj opened this issue Oct 6, 2023 · 1 comment
Labels
auto-triage-skip enhancement M-T: A feature request for new functionality good first issue

Comments

@filmaj
Copy link
Contributor

filmaj commented Oct 6, 2023

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)
@filmaj filmaj added the enhancement M-T: A feature request for new functionality label Oct 6, 2023
Copy link

github-actions bot commented Nov 6, 2023

👋 It looks like this issue has been open for 30 days with no activity. We'll mark this as stale for now, and wait 10 days for an update or for further comment before closing this issue out. If you think this issue needs to be prioritized, please comment to get the thread going again! Maintainers also review issues marked as stale on a regular basis and comment or adjust status if the issue needs to be reprioritized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-triage-skip enhancement M-T: A feature request for new functionality good first issue
Projects
None yet
Development

No branches or pull requests

1 participant