slackapi/hubot-slack

SlackBot#send() support for a callback function

Open

#473 geöffnet am 13. Apr. 2018

Auf GitHub ansehen
 (0 Kommentare) (6 Reaktionen) (0 zugewiesene Personen)CoffeeScript (637 Forks)batch import
enhancementgood first issue

Repository-Metriken

Stars
 (2.339 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Description

in order to know anything about a message after it has been sent, there needs to be a way to run a callback after the chat.postMessage() method completes.

it seems that the Response#send() (from Hubot core) method will invoke the adapter's send() method with the final item in the messages... array set to a function, if the user's script specified one.

this means that all we need to do is pass that callback function into the SlackClient#send() method and make sure it's used as the callback (we can set a default value of a noop function if not).

this support should also be extended to reply() and setTopic().

this would unlock use cases such as #91

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.

Contributor Guide

Research-Richtung
Untersuchen Sie, wie Hubots Response#send() einen Callback als letztes Argument übergibt, und ändern Sie dann SlackClient#send(), reply() und setTopic(), um den Callback zu akzeptieren und mit der Slack API Antwort aufzurufen.
Tech Stack
javascriptnodejs
Domain
backendapideveloper experience
Issue Type
Funktion
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.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Node.jsJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70