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.

    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

      0.3.1

      • Fix issue if extra_text is null
      • Actually create extra_text column

      Updating

      composer require fof/webhooks
      php flarum migrate

      0.3.2

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

      Updating

      composer require fof/webhooks

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

        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 replied to this.

          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

          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

            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!

            datitisev changed the title to FriendsOfFlarum Webhooks .

            Oh! That's a good update and adds 😍

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

            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.

              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 😂

                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! 😀

                  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.

                  Please add new line support for extra text! And something like ${link} so you can customize where the link is! Maybe even add custom embeds! Or add custom webhook support

                  14 days later

                  What does the user dropdown under: "Here you modify what events the webhook is fired on" means?

                  I can chose between admins and everybody else