fix error "You must pass either an array or an instance of Spatie\SlashCommand\AttachmentField" in setFields method.

The error is caused by a incompatible signature on addField method

After installing the package I had to manually copy the config as the vendor:publish command did not work -

php artisan vendor:publish --provider="Spatie\SlashCommand\SlashCommandServiceProvider" --tag="config"
Nothing to publish for tag [config].

I left the config url to slack but when I POST to my domain.app/slack I get route the default NotFoundHttpException in RouteCollection.php line 161:.

Should I be defining any routes for this or does the config itself handle the routing?

I have setup ngrok to use my local homestead app in slack but since this does not work I tried with postman to post directly. Still the same result.

I needed this feature while using the package, and figured I'd extract my custom logic into a contribution back upstream. This feature aims to add the ability to add actions to an attachment, in basically the same fashion as you'd add fields.

Example:

$attachment = Attachment::create()
        ->setColor('good')
        ->setText('This is good!')
        ->addAction(
            AttachmentAction::create('good', 'This is a good button!', 'button')
        )

I still need to add the ability to add menu actions, but those are significantly more involved. With that, I've got a couple of questions for the maintainers.

• Would you like to keep a generic AttachmentAction that has all the abilities of both buttons and menus, or split the two off into a Button and Menu class.
• Would you like to keep the class name AttachmentAction, or instead change it to (FQCN) \Spatie\SlashCommands\Attachment\Action?

Pinging @barryvdh

First of all thanks for all the work you did on this package.

I've opened this issue to discuss a few things regarding your PRs before tagging a new version.

• I've refactored that CatchAll command a bit for readability, could you tests if it works properly in your app?
• in one of the PR's you've added validate to SignatureHandler. Is that method necessary?
• We have some extensive docs on this package. If you have the time, I'd appreciate if you could add some info on your changes. But don't feel obligated to do this in any way.
• On first thought there are now breaking changes, so I would probably tag a new feature version. Have I missed any breaking changes?

First of all, been playing with this all day and its saved me hours of development. Big fan already. 👍

I was looking at ways to change the format of the output as it can become hard to read if there are many items in the response. One way I was thinking was using the Jobs to break up the items into mx 5 responses, giving the output a break in between the lines.

Following on from your tutorial it mentions that you can inject the dependencies into the handle method of the job i.e. -

namespace App\SlashCommand\Jobs;

use Spatie\SlashCommand\Jobs\SlashCommandResponseJob;

class SlowApiJobJob extends SlashCommandResponseJob
{
    // notice here that Laravel will automatically inject dependencies here
    public function handle(MyApi $myApi)
    {
        $response = $myApi->fetchResponse();

        $this
           ->respondToSlack("Here is your response: {$response}")
           ->send();
    }
}

However Laravel fails and complains that Declaration of MyApp\Jobs\LatestPostsJob::handle(MyApp\Contracts\PostRepository $postRepository) must be compatible with Spatie\SlashCommand\Jobs\SlashCommandResponseJob::handle()

I tried to add this to a construct method but I got the following - Symfony\Component\Debug\Exception\FatalThrowableError: Type error: Argument 1 passed to MyApp\Jobs\LatestPostsJob::__construct() must be an instance of MyApp\Contracts\PostRepository, none given

Is there something I am missing?

Would you think of maybe implementing the ability to start sending dialog responses to slack? In conjunction with the existing delayed response setup this would be a great addition to the existing library.

According to https://api.slack.com/slash-commands#best_practices it is best to create a help function. This creates that, for the SignatureHandlers. Requires the signature to be public.

Ideas for improvement:

• Add a command name/description for each handler, like the Console, to show all commands.
• Use the Console Descriptor to generate extensive documentation based on the signature parts.

Overview

I changed the required PHP Version in composer.json to allow PHP 8 by setting the value to ^7.3|^8.0. This should allow the package to work in PHP versions >= 7.3 and >= 8.0.

Note: I am very unsure what branch this should merge to, so I just set it to master 🤷

Testing Steps

• [x] composer install using PHP 7.4 (my old env)
• [x] composer install using PHP 8.0 (my new env)
• [ ] What other tests are run on a release?

I've been following the instructions and I believe I set it all up correctly and my Hodor class works as it should, but when I add a second handler like the Repeat class from examples and register it, the whole program breaks and returns error 500 for both classes.

I would really appreciate it if you could point me in the right direction please as I can't figure out what I am doing wrong, thanks a lot

While following the tutorial, a class not found error is thrown when using the examples that use attachments unless the following is added:

use Spatie\SlashCommand\Attachment; use Spatie\SlashCommand\AttachmentField;

Not sure if this the correct behavior or if something is wrong with my set up. If these should be included, it should note this in the tutorial documentation.

I've just been following your blog post (https://murze.be/2016/07/building-a-slack-bot-with-laravel/) and for fun and to test it - used your Hodor class example.

I followed all your installation steps and everything installed fine - however when Slack posts to my Laravel app I get an error that:

syntax error, unexpected ':', expecting ';' or '{'

Which is referring to ../vendor/spatie/laravel-slack-slash-command/src/Controller.php:31:

public function getResponse(): IlluminateResponse
{
    $this->guardAgainstInvalidRequest();

    $handler = $this->determineHandler();

    try {
        if ($handler instanceof SignatureHandler) {
            $handler->validate();
        }
        $response = $handler->handle($this->request);
    } catch (SlackSlashCommandException $exception) {
        $response = $exception->getResponse($this->request);
    } catch (Exception $exception) {
        $response = $this->convertToResponse($exception);
    }

     return $response->getIlluminateResponse();
}

Am I missing something?