Documentation error? Receiving Illuminate\Http\Exception\HttpResponseException about apiato

[Mahmoudz]

@denizdogan all my requests implement the authorize() and the rules() functions.
I updated the doc to make the authorize() REQUIRED instead of OPTIONAL.
What do you think I should else fix or include in the Documentation?

[denizdogan]

Further investigation shows that if I subclass App\Port\Request\Abstracts\Request, I must also implement authorize. Then it works.

[denizdogan]

@Mahmoudz: Thanks, I feel like we can close this issue now.

In the future development of Hello API, maybe we could refactor some things about the way Request objects work? Too often I have to override the all method, which feels like a hack. It's like URL parameters are second-class citizens compared to "real" parameters, but many routes use only URL parameters! :)

[Mahmoudz]

@denizdogan 100% that needs refactoring, so far the only working solution for that is to hack the all() function. Maybe we can add some nice function in Hello API for that or contribute to the Laravel core with a nicer solution..