mostafamaklad / laravel-permission-mongodb

Associate users with roles and permissions using Laravel and MongoDB
https://maklad.dev
MIT License
112 stars 66 forks source link

Laravel 10 support #150

Closed sloan58 closed 9 months ago

sloan58 commented 9 months ago

Description

Update to mongodb/laravel-mongodb for Laravel 10 support. Refactor Models and Tests.

Motivation and context

Provide package support for Laravel 10

If it fixes an open issue, please link to the issue here (if you write fixes #num or closes #num, the issue will be automatically closed when the pull is accepted.)

How has this been tested?

All included tests are passing in dev

Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code, etc.

Screenshots (if appropriate)

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

sloan58 commented 9 months ago

Tests failed with:

Problem 1
    - mongodb/laravel-mongodb[4.1.0, ..., 4.1.1] require ext-mongodb ^1.15 -> it is missing from your system. Install or enable PHP's mongodb extension.

Do we need to bump the ext-mongodb version for the automated testing env? I'm not sure if/how I can do that.