FriendsOfFlarum Webhooks

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.

datitisev

0.3.1

Fix issue if extra_text is null
Actually create extra_text column

Updating

composer require fof/webhooks
php flarum migrate

DursunCan

FriendsOfFlarum/webhooks24

datitisev

0.3.2

Fix error when deleting posts that have an array object as their content (e.g. discussion rename post)
- Thanks DursunCan

Updating

composer require fof/webhooks

Amarok

Is it possible to post an excerpt and not the entier discussion ?

datitisev

Amarok Not right now. Probably a good setting to have, to limit the text to X characters for post contents.

ItsMrReals

Can you please add filter by tags, we're making a qotd system, and need it to automatically post them, but we don't want it to post all discussions!

ctml

ItsMrReals
+1 for this, would love to filter events by tag so posts, etc. in private discussions can be filtered out. I guess you can still sort of do that with the group permission filter, but the tag just allows for a bit more granularity. Tag filters would let you create webhooks for the same events, and have them go to different locations depending on tag. For example you could have a webhook for a support tag and have it go to your slack channel for support, and announcements to a separate location.

Edit:
FriendsOfFlarum/webhooks25
FriendsOfFlarum/webhooks26

datitisev

0.4.0

NEW: Allow restricting webhook to a single specific tag (only affects Discussion & Post events)
NEW: Allow creating multiple webhooks with same URL
Use User display names everywhere (except User Renamed event)
Add loading indicator for service, url items when being saved
Make sure post contents being sent with webhook aren't an array & post exists
Sightly improve styling of admin page

Updating

composer require fof/webhooks
php flarum cache:clear

ctml

datitisev NEW: Allow restricting webhook to a single specific tag
NEW: Allow creating multiple webhooks with same URL
Use User display names everywhere (except User Renamed event)

You're a rock star, thank you @datitisev!

Amarok

Oh! That's a good update and adds 😍

One more suggestion, adding webhook to Guilded (direct discord concurrent)

ctml

I had to create the tag_id column manually as it was throwing an error when updating my existing webhook. I didn't clear the cache as instructed though so it may have been my fault, but I am not sure that would have any effect or not. Maybe this only affects those who upgrade and not new installs.

Everything with the new features seems to be working smoothly though 🙂 The only other important ones might be sanitizing bbcode/markdown from posts (optional) and a more minor would be to truncate posts after X characters (optional) with a message for the user "Click here to read the full post" sort of like flarum email notifications.

datitisev

ctml I had to create the tag_id column manually as it was throwing an error when updating my existing webhook

Did you run php flarum migrate? If not, next time your forum tries to migrate it will have an error because it'll try to create the column that you've already created.

ctml The only other important ones might be sanitizing bbcode/markdown from posts (optional)

I attempted to do this, and code blocks had an ugly script tag that wasn't getting removed for some reason so I gave up for today 😂

ctml

DursunCan did you ever create this column manually (like I did a long time ago), or did you run the migrate after extension updated?

datitisev
ctml

You could rename the column temporarily, tag_id_temp, run the migrate, enable extension, if all good then remove the newly created tag_id column and then rename the old one back to tag_id.

ctml

datitisev Did you run php flarum migrate? If not, next time your forum tries to migrate it will have an error because it'll try to create the column that you've already created.

Ah ok, no I did not do that! my fault 😁 I will make sure to remove the column and run so it doesn't cause issues later, thank you for the heads up.

datitisev I attempted to do this, and code blocks had an ugly script tag that wasn't getting removed for some reason so I gave up for today 😂

The triple grave accent ``` for large code blocks are working for slack since the two using the same markdown but single and double won't as they are not a part of slack mrkdwn. I guess it would be more work than its worth to try and convert bbcode/markdown or html to slack specific mrkdwn unless someone already had a reliable library that did it. For now in my posts I am just going to avoid using any bbcode/markdown that I know will trigger webhooks so things look pretty 🙂 Thanks for having a look into the idea! 😀

datitisev

ctml Well, it's not about the markdown - I tried using a library to convert the rendered HTML into Markdown but TextFormatter adds a JS script for the syntax highlighting that didn't want to be removed. I'll look into it some other time, but it's not a priority to me.

« Previous Page Next Page »