Closed nes0 closed 5 months ago
It's hard to tell without testing it but it should be ok, it can also be wrapped in an exception try/catch block in case there is an error.
Do you have the value for the environment variable DISCORD_WEBHOOK
?
Thanks Arthur. Yes I have a value for DISCORD_WEBHOOK. I don't have access to the production environment though. I'm hoping someone else can help with that. Shall I email it to you?
I've just added an exception handler which should log in the event of any obvious failure.
DMed you the hook
Hey guys, the key has been added to the environment file and should be good to go
Thanks Arthur. Shall we deploy this change and see if it works?
Looks good
Hopefully this change replaces UserObserver Slack posts with a post to a Discord webhook URL which will be retrieved from the production environment. I don't know PHP or Laravel, so would really appreciate it if you would check this over for correctness. I've tried to keep it as simple as possible.