Skip to content

Include an example of using middleware with the ExpressReceiver#1973

Merged
zimeg merged 2 commits intoslackapi:mainfrom
zimeg:middleware-docs
Oct 18, 2023
Merged

Include an example of using middleware with the ExpressReceiver#1973
zimeg merged 2 commits intoslackapi:mainfrom
zimeg:middleware-docs

Conversation

@zimeg
Copy link
Copy Markdown
Member

@zimeg zimeg commented Oct 16, 2023

Summary

This PR adds an example of using middleware in web requests with the ExpressReceiver to the custom routes documentation.

Notes

  • As always, very open to any suggestions, changes, removals, or additions.
  • Any help with the translations would also be very appreciated!

Requirements

@zimeg zimeg added docs M-T: Documentation work only semver:patch labels Oct 16, 2023
@zimeg zimeg self-assigned this Oct 16, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 16, 2023

Codecov Report

Merging #1973 (9648add) into main (f104c0b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1973   +/-   ##
=======================================
  Coverage   82.21%   82.21%           
=======================================
  Files          18       18           
  Lines        1524     1524           
  Branches      438      438           
=======================================
  Hits         1253     1253           
  Misses        175      175           
  Partials       96       96           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more


(async () => {
await app.start();
console.log('⚡️ Bolt app started'');
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch 👍

Copy link
Copy Markdown
Contributor

@seratch seratch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@seratch seratch added this to the 3.14.1 milestone Oct 17, 2023
Copy link
Copy Markdown
Contributor

@WilliamBergamin WilliamBergamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zimeg
Copy link
Copy Markdown
Member Author

zimeg commented Oct 18, 2023

@seratch @WilliamBergamin Appreciate the reviews, thank you! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs M-T: Documentation work only semver:patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants