Skip to content

Fix #80 by making app.action/options more flexible#83

Merged
seratch merged 1 commit into
slackapi:mainfrom
seratch:issue-80-actions-options
Sep 12, 2020
Merged

Fix #80 by making app.action/options more flexible#83
seratch merged 1 commit into
slackapi:mainfrom
seratch:issue-80-actions-options

Conversation

@seratch

@seratch seratch commented Sep 12, 2020

Copy link
Copy Markdown
Contributor

This pull request fixes #80 for both app.action and app.options for better compatibility with Bolt for JS.

Category (place an x in each of the [ ])

  • slack_bolt.App and/or its core components
  • slack_bolt.async_app.AsyncApp and/or its core components
  • Adapters in slack_bolt.adapter

Requirements (place an x in each [ ])

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

  • 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 run ./scripts/run_tests.sh after making the changes.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #83 into main will increase coverage by 0.11%.
The diff coverage is 97.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   82.60%   82.71%   +0.11%     
==========================================
  Files         128      128              
  Lines        3535     3553      +18     
==========================================
+ Hits         2920     2939      +19     
+ Misses        615      614       -1     
Impacted Files Coverage Δ
slack_bolt/listener_matcher/builtins.py 89.18% <97.29%> (+1.76%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba14b1f...c4a8065. Read the comment docs.

@seratch seratch merged commit a65c125 into slackapi:main Sep 12, 2020
@seratch seratch deleted the issue-80-actions-options branch September 12, 2020 08:00
@seratch seratch mentioned this pull request Sep 14, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

app.action without type should work for more actions

2 participants