FriendsOfFlarum Webhooks

datitisev

ctml That's currently available in the pro version (reflar/webhooks-pro) on Extiverse. I don't think I ever made a discussion for it here. The problem with that is that it's not the same as an API response, so it has weird logic and sometimes doesn't work completely depending on the event and the payload.

ajhalili2006

I got an important message from Discord (you know it, I'm using their APIs) about the API domain migration last July 18. On November 7, they'll be dropping support for the discordapp.com in favor of discord.com and if the plugin still asks for the discordapp.com domain and attempt to send requests beyond that date, errors will fire up in the console.

Related Issue: ReFlar/webhooks21

EvilExecutor

Hey! When I enable the extension the discussions/post won't get posted at once. It asks for a reload. Once the reload is done the discussions/post gets posted and the message is sent to discord. But when I disable the extension everything works fine... Have I done something wrong?

datitisev

EvilExecutor There should be an error in the Flarum log file at storage/logs/flarum-<date>.log

EvilExecutor

Nope there isn't 🙁

clarkwinkelmann

You can also enable debug in config.php and try again to have the error displayed on screen.

If it's not logged, it's either a javascript error instead of backend error. Or it's a backend error and the log folder isn't writable.

EvilExecutor

clarkwinkelmann
Sounds like Greek to me 😅

datitisev

EvilExecutor That's... interesting... and doesn't really show where the error comes from. At a glance it doesn't seem like its' an issue with webhooks but a dependency of it. 🤔

clarkwinkelmann

EvilExecutor seems like this is the same issue googleapis/google-api-php-client1392, https://stackoverflow.com/q/50898330

You could try running composer update guzzlehttp/guzzle --with-dependencies to update Guzzle to the latest possible version for your setup and see whether the issue goes away.

From those linked threads, it seems updating to Guzzle 6.3 or higher is necessary to fix the issue. You can use composer show guzzlehttp/guzzle --tree to check which version of Guzzle and its dependencies are installed.

EvilExecutor

composer require guzzlehttp/guzzle:^6.3 made it work as charm (was a blind shot for me) 🙂 Thanks @clarkwinkelmann

clarkwinkelmann

EvilExecutor Good news! However I wouldn't recommend using require, because now it's hard-coded in your composer.json and could prevent an update later on when all extensions move to Guzzle version 7.

You can simply remove the line about guzzle in your composer.json to fix this problem.

ItsMrReals

Is there any support for by tag?

Like so it only sends the webhook if it had a tag.

datitisev

ItsMrReals Not at the moment. That will probably be added in the future.

EvilExecutor

Any plans to update this extension?

datitisev

EvilExecutor Yes. I can't provide a timeframe for it, however. Sorry 😕

EvilExecutor

datitisev no problem take your time!

idkwhatusername

Can you please update it to beta-14?

Amarok

Hope an update will come those days 🙏

ctml

Will this come to beta 15?

Amarok

datitisev EvilExecutor Yes. I can't provide a timeframe for it, however. Sorry 😕

ctml 😉 maybe, but not in priority

« Previous Page Next Page »