Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow spatie laravel permissions v5.0 #167

Closed
wants to merge 1 commit into from

Conversation

SiebeVE
Copy link
Contributor

@SiebeVE SiebeVE commented Sep 3, 2021

No description provided.

@lloricode
Copy link
Contributor

1+

@lloricode
Copy link
Contributor

before merge this, this needs an update, since spatie permission v5 has team management, and also a BC breaks

@lloricode
Copy link
Contributor

Its seem no BC break from v4 to v5, it can be add support to v5 first, then later add feature of team manage with spatie permission v5

@lloricode
Copy link
Contributor

i try to install with spatie permission v5, it works great!

   "require": {
       "spatie/laravel-permission": "^5.0",
        "vyuldashev/nova-permission": "dev-master",
     },
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/SiebeVE/nova-permission.git"
        }
    ]
}

@ralphschindler
Copy link

@vyuldashev looking forward to it working with v5! 👍 😃

@niekbr
Copy link

niekbr commented Mar 7, 2022

Would appreciate a release :)

@Ahmedd-Ibrahim
Copy link

I made a temporary solution until they look into this,
I forked it and made it in a separate package
https://github.com/Ahmedd-Ibrahim/nova-permission

@jamespavett
Copy link

I made a temporary solution until they look into this,

I forked it and made it in a separate package

https://github.com/Ahmedd-Ibrahim/nova-permission

Nice but if you look at #174 you will see this was already done by someone else.

@parallels999
Copy link

Fixed on 891d1c9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants