Inconsistent SlackObjectFormationError raising when empty text is provided.
Les mainteneurs répondent en général sous 5 jours
@hello-ashleyintech y travaille déjà.
Depuis le 20/9/2022.
Évaluation
Cette issue n'a pas encore été évaluée.
Description
Given the following:
>>> SectionBlock(text="").to_dict()
SlackObjectFormationError: text or fields attribute must be specified
This is all correct enough and good enough. What's happening behind the scenes appears to be that TextObject.parse is used and does a truthy test on the incoming text, and returns None as opposed to an empty string.
Where that begins to go wrong, is here:
>>> SectionBlock(text=MarkdownTextObject(text="")).to_dict()
{'text': {'type': 'mrkdwn'}, 'type': 'section'}
Slack will reject that as erroneous. If it's sent back as part of the acknowledgement response as a response_action we won't necessarily see (receive) an error related to failing to parse the JSON schema. Slack's block kit builder says of it: Missing required field: text
Category
- slack_sdk.models (UI component builders)
Desired outcome
Ideally, I'd like to be able to trust (more) that the validation present in slack_sdk is going to raise on invalid configurations early, and consistently -- that is, where possible I'd like to assume that putting together something "empty" even if I've opted to use the classes instead of an equivalent base type (str, dict) should raise.
- Langage dominant
- Python
- Étoiles
- 4k
- Forks
- 861
- Merge moyen
- 3 j 17 h
- PR mergées (30 j)
- 10
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de slackapi/python-slack-sdk
-
Missing method from SDK // admin.users.getExpirationPeut-être pris @srtaalej l’a pris il y a 3 jours. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
slackapi/python-slack-sdk#1982 · 1 commentaire ·
Les mainteneurs répondent en général sous 5 jours
-
auto-triage-stale needs info server-side-issue
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
slackapi/python-slack-sdk#1961 · 4 commentaires ·
Les mainteneurs répondent en général sous 5 jours
-
auto-triage-skip discussion
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
slackapi/python-slack-sdk#1940 · 2 commentaires ·
Les mainteneurs répondent en général sous 5 jours
-
chat_postMessage silently forwards thread_id to the API, so a threaded reply posts to the channelOuverteauto-triage-skip enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
slackapi/python-slack-sdk#1923 · 2 commentaires ·
Les mainteneurs répondent en général sous 5 jours
-
files_upload_v2 should respect the retry handlersPeut-être pris @DSeaStar l’a pris il y a 59 jours. Ouverteauto-triage-skip bug python web-client
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
slackapi/python-slack-sdk#1853 · 2 commentaires ·
Les mainteneurs répondent en général sous 5 jours
Toutes les issues de slackapi/python-slack-sdk
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 87/100
adri0/scadustats#128 ·
Les mainteneurs répondent en général sous 1 jour
-
`TIMESTAMP`, `TIMESTAMP_NTZ`, `TINYINT` and the other `TypeDecorator`s don't declare `python_type`Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
RPBCOuverteawaiting policy acceptance precheck-passed
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
Bioconductor/BiocContributions#240 · 1 commentaire ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
documentation good first issue launch
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
FlologixAI/etchwork#4 ·