FriendsOfFlarum Webhooks

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!

Amarok

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

ctml 😉 maybe, but not in priority

idkwhatusername

Can you please update it to beta-14?

Amarok

Hope an update will come those days 🙏

ctml

Will this come to beta 15?

datitisev

0.3.0

This extension does not support beta 14. Your ReFlar webhooks will be migrated.

Moved to FriendsOfFlarum
Updated for beta 15
Fix issue with all events in Webhook settings modal showing disabled animation every time you enable/disable one
Fix default group showing as member
Improve settings UI
NEW: Add extra text to your webhook messages (predefined, cannot change per event, set per webhook)
NEW: Microsoft Teams integration
DISCORD: Do not replace avatar with forum icon
DISCORD: Support discordapp.com and discord.com domains
- A migration on extension enable will automatically replace discord.com with discordapp.com from all your Discord webhook urls

Updating

composer remove reflar/webhooks
composer require fof/webhooks

ctml

datitisev

Happy holidays, thank you for this nice gift!

Amarok

datitisev Thx for this upgrade 🥰

JasperVriends

datitisev Awesome! Is the extra text required? Because without you get the error using the Discord webhook. Otherwise I'll open an issue in GitHub

Return value of FoF\Webhooks\Response::getExtraText() must be of the type string, null returned

Also, when trying to add the extra text you'll get an error (that's a bug for sure)

datitisev

JasperVriends What error do you get when trying to add the extra text?

I will fix the no extra text error today (null one( but i didn't encounter an error when updating the extra text.

JasperVriends

datitisev Ah! I see the issue, the column extra_text does not exists after migrating from 'Webhooks by ReFlar' to 'FriendsOfFlarum webhooks'. There is no migration that adds the extra_text culumn as the 'create table' migration hasn't run due to the table already exists

datitisev

JasperVriends Oh right, i forgot that i never actually released a version with that column. I made changes back in June but never published them 🙈. Will fix soon.

« Previous Page Next Page »