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
Function to() doesn't work #54
Comments
You mention creating an app but we don't support that atm. Only incoming webhooks. |
Hmm, I'll try to have a look at this to see if we can support apps as well. For now I don't think this will work sorry. |
Hi Dries - not looking to reopen this, however, the doco mentions that we need to create a slack app for the integration to work, so yeah, we create an app. Does the doco needs to be updated? I might have been following them a little too closely if creating an app is not required! Doco screengrab: The slack documentation indicates that we're (nowadays) unable to send incoming webhooks without setting up a slack app |
Hey @ok200paul, you're correct. It seems we updated those docs incorrectly. I've sent in a PR to revert this as this currently doesn't work: laravel/docs#7651. Like I said, I'll be looking into modernising the package hopefully soon so Slack Apps are supported as well. |
Thanks Dries! Ping me when you face back in this direction and I’ll help
out where I can.
On Fri, 4 Feb 2022 at 19:07, Dries Vints ***@***.***> wrote:
Hey @ok200paul <https://github.com/ok200paul>, you're correct. It seems
we updated those docs incorrectly. I've sent in a PR to revert this as this
currently doesn't work: laravel/docs#7651
<laravel/docs#7651>.
Like I said, I'll be looking into modernising the package hopefully soon
so Slack Apps are supported as well.
—
Reply to this email directly, view it on GitHub
<#54 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFK6K63PAEVM4R4GUKVQR6TUZOCMDANCNFSM5NKMD3JA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Regards,
Paul Grimes
OK200
129a Chapel St,
Windsor 3181
Melbourne
P: 0432 903 145
E: ***@***.***
https://ok200.net*Looking to set up a meeting?* Check my availability here:
https://calendly.com/ok200-paul/30min
|
@ok200paul we've decided to leave the current docs in place and just note the methods that don't work with Slack apps. It looks that you can't use |
Will do, thanks Dries! |
Description:
Specifying an alternative channel to send a slack notification to, does not work. Probably linked to #34 from back in 2020.
Steps To Reproduce:
Demo
https://github.com/ok200paul/l8-slack-channel-bug-report
The text was updated successfully, but these errors were encountered: