Can't seem to send notifications to Slack

[kaka-ruto]

Hi, first thanks for the gem!

I am using noticed -v 2.4.3, rails 7.2, solid queue 0.8.0 and I have this notifier I intend to use to send messages to Slack channels

# To deliver this notification:
#
# CarModel::GeneratorNotifier.with(message: "message").deliver_later(Current.system_account)

class CarModel::GeneratorNotifier < ApplicationNotifier
  deliver_by :slack do |config|
    config.url = -> {
      if params[:error]
        Rails.application.credentials.dig(:slack, :errors_webhook_url)
      else
        Rails.application.credentials.dig(:slack, :wins_webhook_url)
      end
    }
    config.json = -> {
      {
        blocks: [
          {
            type: "section",
            text: {
              type: "mrkdwn",
              text: params[:message]
            }
          },
          {
            type: "context",
            elements: [
              {
                type: "mrkdwn",
                text: "Environment: #{Rails.env}"
              }
            ]
          }
        ]
      }
    }
  end

  # If you need to perform any additional setup or have helper methods,
  # you can define them here

  # Example of a helper method
  def message
    params[:message]
  end
end

The first hurdle I had to figure out was to include a default recipient, as It was not getting far without one.

But then after that, the job is enqueued but nothing get's sent to Slack.

This is the result object I get from calling the notifier on rails console

#<CarModel::GeneratorNotifier:0x000000012561e5b0
 id: 72,
 account_id: 2,
 type: "CarModel::GeneratorNotifier",
 record_type: nil,
 record_id: nil,
 params: {:message=>"message"},
 created_at: "2024-09-12 03:08:52.788541000 +0000",
 updated_at: "2024-09-12 03:08:52.788541000 +0000",
 notifications_count: 1>

Anything I need to do to send messages to a Slack channe?

[excid3]

You should use a bulk delivery since this doesn't need any recipients.

Delivery enqueues a job, so make sure your jobs are being processed and check them for errors.

[kaka-ruto]

Thanks Chris, for bulk deliveries, I need to set it this way? deliver_by :slack, bulk: true do |config|

[excid3]

Should be bulk_deliver_by. Our docs are wrong on those so I just updated them.

[kaka-ruto]

Thanks Chris