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

Upgrade to spatie/laravel-permission v6.1 #347

Closed
callmejed opened this issue Dec 7, 2023 · 5 comments
Closed

Upgrade to spatie/laravel-permission v6.1 #347

callmejed opened this issue Dec 7, 2023 · 5 comments

Comments

@callmejed
Copy link

I'm using Spatie ^6.1 so composer require fails since this package seems to require Spatie ^5.0

Copy link

welcome bot commented Dec 7, 2023

Hello there! Thanks for opening your first issue on this repo!

Just a heads-up: Here at Backpack we use Github Issues only for tracking bugs. Talk about new features is also acceptable. This helps a lot in keeping our focus on improving Backpack. If you issue is not a bug/feature, please help us out by closing the issue yourself and posting in the appropriate medium (see below). If you're not sure where it fits, it's ok, a community member will probably reply to help you with that.

Backpack communication mediums:

  • Bug Reports, Feature Requests - Github Issues (here);
  • Quick help (How do I do X) - Gitter Chatroom;
  • Long questions (I have done X and Y and it won't do Z wtf) - Stackoverflow, using the backpack-for-laravel tag;

Please keep in mind Backpack offers no official / paid support. Whatever help you receive here, on Gitter, Slack or Stackoverflow is thanks to our awesome awesome community members, who give up some of their time to help their peers. If you want to join our community, just start pitching in. We take pride in being a welcoming bunch.

Thank you!

--
Justin Case
The Backpack Robot

@promatik
Copy link
Contributor

promatik commented Dec 8, 2023

Hi @callmejed!

I've read the upgrade guide and it seems there was a lot of changes/possible breaking changes.
https://github.com/spatie/laravel-permission/blob/main/docs/upgrading.md#upgrading-from-v5-to-v6

We're probably work this out for the new backpack version.

@promatik promatik changed the title Can this package be updated to use current dependencies? Upgrade to spatie/laravel-permission v6.1 Dec 8, 2023
@tabacitu
Copy link
Member

tabacitu commented Dec 8, 2023

cat-glasses-gif-by-leroy-patterson

We should be able to add support for spatie/laravel-permission v6, it has nothing to do with backpack/crud. We can do this:

  • backpack/permissionmanager v7 will support spatie/laravel-permission v3, v4, v5
  • backpack/permissionmanager v8 will support only spatie/laravel-permission v6

So yeah, I'd stay we should prioritize and do this, if people use the latest PermissionManager they can't add this Backpack add-on on top.

@callmejed
Copy link
Author

That would be fantastic <3

@promatik
Copy link
Contributor

promatik commented Apr 7, 2024

This one was fixed by @pxpm 3 weeks ago 🎉
#356

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

No branches or pull requests

3 participants