slackapi/python-slack-sdk

Improve structure of `admin.apps.config.set` HTTP API arguments

Open

#1.412 geöffnet am 6. Okt. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (3.738 Stars) (885 Forks)batch import
auto-triage-skipenhancementgood first issue

Beschreibung

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

Tech Stack
python
Domain
apideveloper experience
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
PythonSlack APIPython Slack SDK basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
Familiarize yourself with the improved argument types from the referenced node slack sdk PR (slackapi/node slack sdk#1676). Then, locate the `admin.apps.config.set` method in the Python SDK, likely within the `slack sdk/web` module. Adapt the TypeScript types to Python type hints, ensuring consistency with the SDK's existing patterns. Review any comments in the issue for additional guidance.