Optionales Verketten von Beiträgen zum übergeordneten Thema in der Slack-Integration

[mcdanlj (Michael K Johnson)]

Ich habe den PR schon stark bereinigt, aber er ist noch nicht ganz bereit für den Merge. Ich habe bisher zwei oder drei Dinge, die mir Schwierigkeiten bereiten, und ich weiß noch nicht, wie ich sie beheben soll.

1. Ich versuche, fa-arrow-circle-o-right als Thread-Icon zu verwenden, aber es wird in der UI auf meiner Live-Website leer angezeigt. (Nach dem Auschecken meines Branches auf der Live-Website führe ich su discourse -c 'bundle exec rake assets:precompile' && sv restart unicorn aus, um es auf dem Live-Server zu testen.) Ich habe es sowohl in plugin.rb hinzugefügt als auch referenziert, aber ich bin ratlos, was die nächsten Schritte sein sollen. Gibt es eine Liste der Font Awesome-Icons, die für die Verwendung in Discourse genehmigt sind? Ich habe lib/svg_sprite/svg_sprite.rb gefunden, und chevron-right sieht für diesen Zweck großartig aus.

2. Die Tests laufen bei mir lokal alle durch, aber in Travis erhalte ich konsistente Fehler, die anscheinend nichts mit meinen Änderungen zu tun haben. Das macht es für mich natürlich schwer, sie zu untersuchen oder zu analysieren. 13 Fehler mit einem 404 statt eines erwarteten Codes (z. B. 200) in spec/lib/discourse_chat/provider/slack/slack_command_controller_spec.rb Behoben, indem ich isolate_namespace nicht mehr blind nachgebaut habe, und jetzt kenne ich rake routes.

Ich habe erfolgreich gepostet:

Es könnte noch etwas zu bereinigen geben, aber ich denke, das funktioniert.

Nach dem Merge werde ich Discourse Chat Integration entsprechend aktualisieren.