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

For API? #1

Open
shirshak55 opened this issue Sep 30, 2017 · 6 comments
Open

For API? #1

shirshak55 opened this issue Sep 30, 2017 · 6 comments

Comments

@shirshak55
Copy link

I guess there is namespacing on this spatie permissions package. How should we work this in API. As laravel supports Resources in laravel 5.5 the demo should not be hard for api? For Api are we going to duplicate soo much roles and permissions in tables?? Can we make demo for that also as in modern days there will be mobile app version of almost every website right?

@drbyte
Copy link
Owner

drbyte commented Sep 30, 2017

Adding a demo of the API Resource support in 5.5 is a good idea.
Willing to collaborate to put one together. Wanna help?

I'd like to address your namespacing question, but I think it's better handled over in the package repo itself, as it relates to the multiple guards feature. Do you mind posting this as an Issue over in the https://github.com/spatie/laravel-permission/issues/ repo?

@shirshak55
Copy link
Author

Hi I am willing to collabrate . Yes I guess we need to discuss with freekmurze. I guess namespacing problem will be fixed on v3 of that package.

spatie/laravel-permission#479

@drbyte
Copy link
Owner

drbyte commented Sep 30, 2017

Great!

If you're okay with the demo leveraging what's already here in this demo, shall we just do it right here, using PRs?

I'll clone master into api as a working branch, and when we're satisfied it's complete we can merge into master.

@shirshak55
Copy link
Author

Sure . We will work in api and we will leverage whats already here as it will maintain consistency

@drbyte
Copy link
Owner

drbyte commented Sep 30, 2017

Excellent.

@eternalBlast
Copy link

Is this role for api can be used now?

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

No branches or pull requests

3 participants