4261814162
- Stop using `break` in a block - it doesn't work. The failure here was hidden because it was within a `defer` block, so would not cause a server error - Refactor the error handling so that the error is passed back to Slack and displayed to the user - Return specific error messages for user / message-history / message errors - Tidy up the SlackCommandController to make all non-requestable methods private - Add a test to ensure error messages are passed correctly to Slack |
||
|---|---|---|
| .. | ||
| slack_command_controller.rb | ||
| slack_enabled_setting_validator.rb | ||
| slack_message.rb | ||
| slack_message_formatter.rb | ||
| slack_provider.rb | ||
| slack_transcript.rb | ||