authenticated user "user1" cannot access acl-test/basic-authenticated route about laravel-enterprise-starter-kit HOT 4 CLOSED

I guess the error was self-explanatory.. There was indeed no valid permission associated with the aforementioned route, and all the other testing routes. I was expecting this was established by default, but I was wrong.

My question now is: Did you also unintentionally miss setting permission_id to 8 for admin/audit/{userId}/show as with admin/audit ?

from laravel-enterprise-starter-kit.

Hello,
I just double checked and you are correct on both counts. The routes under "acl-test/*" and the route "admin/audit/{userId}/show" do not have permissions assigned to them. That was an oversight sorry about the trouble.
I also noted that the parameter in the audit route definition should be "auditId" instead of "userId", a little too much cut-&-paste if you ask me...

I am currently working on the crumb-trail and dynamic menu based on permissions, and I will make sure to fix this in the next patch.

Thanks for creating the issue.
/s

from laravel-enterprise-starter-kit.

Dear Sebastian,

I'm eager to see your next patch, at least including the crumb-trail and dynamic menu based on permissions, so I don't have to reinvent the wheel. :-)

from laravel-enterprise-starter-kit.

Fixed in commit 30d696f.

from laravel-enterprise-starter-kit.