Skip to content

Commit

Permalink
added migration guide for V2
Browse files Browse the repository at this point in the history
  • Loading branch information
stevengill committed Mar 25, 2020
1 parent 17d23d1 commit 6bcc8aa
Show file tree
Hide file tree
Showing 6 changed files with 129 additions and 3 deletions.
4 changes: 2 additions & 2 deletions docs/_advanced/error_handling.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ order: 1
---

<div class="section-content">
If an error occurs in a listener, it’s recommended you handle it directly. However, there are cases where errors may be triggered after your listener has already returned (such as in `say()`, `respond()`, or if you don't call `ack()` when it's required). By default, these errors will be logged to the console. To handle them yourself, you can attach a global error handler to your app with the `error(fn)` method.
*Note: Since `Bolt@2.x`, error handling has improved! View the [migration guide for V2](https://slack.dev/bolt/tutorial/migration-v2) to learn more.*

If you want more control over errors, it’s advised to use the [`chat.postMessage`](https://api.slack.com/methods/chat.postMessage) method attached to your app under the `client` key (instead of `say()` or `respond()`). This returns a `Promise` that can be caught to handle the error.
If an error occurs in a listener, it’s recommended you handle it directly with a `try`/`catch`. However, there still may be cases where errors slip through the cracks. By default, these errors will be logged to the console. To handle them yourself, you can attach a global error handler to your app with the `error(fn)` method.
</div>

```javascript
Expand Down
2 changes: 2 additions & 0 deletions docs/_advanced/middleware_global.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ Global middleware is run for all incoming events before any listener middleware.
Both global and listener middleware must call `await next()` to pass control of the execution chain to the next middleware, or call `throw` to pass an error back up the previously-executed middleware chain.

As an example, let's say your app should only respond to users identified with a corresponding internal authentication service (an SSO provider or LDAP, for example). You may define a global middleware that looks up a user record in the authentication service and errors if the user is not found.

*Note: Since `Bolt@2.x`, global middleware was updated to support `async` functions! View the [migration guide for V2](https://slack.dev/bolt/tutorial/migration-v2) to learn more.*
</div>

```javascript
Expand Down
1 change: 1 addition & 0 deletions docs/_advanced/middleware_listener.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ But of course, you can write your own middleware for more custom functionality.

As an example, let’s say your listener should only deal with messages from humans. You can write a listener middleware that excludes any bot messages.

*Note: Since `Bolt@2.x`, listener middleware was updated to support `async` functions! View the [migration guide for V2](https://slack.dev/bolt/tutorial/migration-v2) to learn more.*
</div>

```javascript
Expand Down
1 change: 1 addition & 0 deletions docs/_basic/listening_actions.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ Actions can be filtered on an `action_id` of type string or RegExp object. `acti

You’ll notice in all `action()` examples, `ack()` is used. It is required to call the `ack()` function within an action listener to acknowledge that the event was received from Slack. This is discussed in the [acknowledging events section](#acknowledge).

*Note: Since `Bolt@2.x`, message shortcuts (previously message actions) now use the `shortcut()` method instead of the `action()` method. View the [migration guide for V2](https://slack.dev/bolt/tutorial/migration-v2) to learn more.*
</div>

```javascript
Expand Down
2 changes: 1 addition & 1 deletion docs/_tutorials/hubot_migration.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
title: Migrating Hubot apps
order: 1
order: 2
slug: hubot-migration
lang: en
layout: tutorial
Expand Down
122 changes: 122 additions & 0 deletions docs/_tutorials/migration_v2.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,122 @@
---
title: Migrating to V2
order: 1
slug: migration-v2
lang: en
layout: tutorial
permalink: /tutorial/migration-v2
---
# Migrating to v2.x

<div class="section-content">
This guide will walk you through the process of updating your app from using `bolt@v1.x` to `bolt@2.x`.
</div>

---

## Upgrading your listeners to `async`

Listeners in your app should updated to `async` functions and `say()`, `respond()`, and `ack()` should be prefaced with `await`.

Before:

```javascript
app.action('some-action-id', ({action, ack, say}) => {
ack();
say('hello world');
})
```

After:

```javascript
app.action('some-action-id', async ({action, ack, say}) => {
await ack();
await say('hello world');
})
```


## Error Handling

The recent changes in Bolt for Javascript V2 have improved our ability to catch errors and filter them to the global error handler. It is still recommended to manage errors in the listeners themselves instead of letting them propagate to the global handler when possible.

### Handling Errors in Listeners with `try`/`catch`

```javascript
app.action('some-action-id', async ({action, ack, say}) => {
try {
await ack();
await say('hello world');
} catch (error) {
console.log(error)
// handle error
}
})
```

### Handling Errors with the Global Error Handler

```javascript
app.error((error) => {
// Check the details of the error to handle cases where you should retry sending a message or stop the app
console.error(error);
});
```

Other error related changes include:

- When your listener doesn’t call `ack` within the 3 section time limit, we log the failure instead of throwing an error.
- If you have multiple errors at once when running middleware, Bolt for Javascript will return a wrapper error with a `code` parameter of `slack_bolt_multiple_listener_error` and an `original` parameter that contains an array of all of the errors.


## Message Shortcuts

[Message shortcuts](https://api.slack.com/interactivity/shortcuts/using#message_shortcuts) (previously referred to as message actions) now use the `shortcut()` method instead of the `action()` method.

Before:

```javascript
app.action('message-action-callback', ({action, ack, context}) => {
ack();
// Do stuff
})
```

After:

```javascript
app.shortcut('message-action-callback', async ({shortcut, ack, context}) => {
await ack();
// Do stuff
})
```

## Upgrading Middleware

If you wrote a custom middleware, adjust your function to `async` and update `next()` to `await next()`. If your middleware does some post processing, instead of passing a function to `next()`, you can now run it after `await next()`.

Before:

```javascript
function noBotMessages({ message, next }) {
function doAfter() {
// Post processing goes here
}

if (!message.subtype || message.subtype !== 'bot_message') {
next(doAfter);
}
}
```

After:

```javascript
async function noBotMessages({ message, next }) {
if (!message.subtype || message.subtype !== 'bot_message') {
await next();
// Post processing goes here
}
}
```

0 comments on commit 6bcc8aa

Please sign in to comment.