-
Notifications
You must be signed in to change notification settings - Fork 248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add org-level installation support #148
Conversation
100aaae
to
a63931f
Compare
a63931f
to
d78684f
Compare
304e850
to
e6c2a0d
Compare
# installation_or_bot: Optional[Bot] | Optional[Installation] | ||
|
||
|
||
# TODO |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't been able to test this yet since I can't get my local OAuth flow to work (i think my setup is messed up). Even without these changes it is hanging. I will try to sort that out but wanted to post this here so you can build upon it @seratch
Codecov Report
@@ Coverage Diff @@
## main #148 +/- ##
==========================================
- Coverage 91.79% 91.66% -0.14%
==========================================
Files 149 149
Lines 4340 4401 +61
==========================================
+ Hits 3984 4034 +50
- Misses 356 367 +11
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am confident enough to merge this PR now.
(Describe the goal of this PR. Mention any related Issue numbers)
slackapi/python-slack-sdk#877
Category (place an
x
in each of the[ ]
)slack_bolt.App
and/or its core componentsslack_bolt.async_app.AsyncApp
and/or its core componentsslack_bolt.adapter
/docs
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.
./scripts/install_all_and_run_tests.sh
after making the changes.