Mike Rockett Posted October 22, 2017 Share Posted October 22, 2017 @szabesz - I'd prefer it to be inline for quick access to the information (I also think its handy for beginners). I'll add a link to the applicable page though to learn more and see the actual license text. Licenses don't often change (if at all), and it's not all that often that new ones get added to the list. The commits to their license files are only related to URLs, examples, and small fixes. Some new licenses were added, and I can easily keep track of those. But, I won't be linking it up to their repo because some licenses are just not applicable, and so they're excluded from the list. PS, as much as this is going to be a production-product that I'll maintain, it's also a 'playground' of sorts for me to learn Vue. 1 Link to comment Share on other sites More sharing options...
Mike Rockett Posted November 12, 2017 Share Posted November 12, 2017 The current generator uses an array for “permission”, which doesn’t seem right as the API docs state it needs to be a string… Which is correct? Also, if a permission is created via the “permissions” array, surely they would all need to be in “permission” as an array as well? This how I’m doing it in Jumplinks, anyway (hope I’m not wrong). Or perhaps “permissionMethod” should be used for multiple permissions? Some insight would be appreciated. 1 Link to comment Share on other sites More sharing options...
LostKobrakai Posted November 12, 2017 Share Posted November 12, 2017 PW ultimately calls $user->hasPermission() with the supplied permission info, which does expect a single permission to be used. So you'd use the permissionMethod for multiple ones. 1 Link to comment Share on other sites More sharing options...
Mike Rockett Posted November 12, 2017 Share Posted November 12, 2017 Thanks, figured as much. Will need to rework this on the front end then. 1 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now