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

Updating for 5.4 and adding customizations #208

Merged
merged 4 commits into from
Mar 27, 2017
Merged

Updating for 5.4 and adding customizations #208

merged 4 commits into from
Mar 27, 2017

Conversation

stygiansabyss
Copy link
Contributor

  • Updated for Laravel 5.4.
  • Migrations run locally instead of being published.
  • Config option to specify database table prefix added.
  • Config option to detail non-standard users table name added.
  • can() renamed to hasPermission().
  • is() renamed to isRole().

Method renaming was due to conflicts with laravel 5.4 base code.

@kodeine
Copy link
Owner

kodeine commented Mar 24, 2017

following checks failed? can you please fix those so i can merge this pull?

image

thanks

@stygiansabyss
Copy link
Contributor Author

I did fix it in the following push. 33dfe6e

@kodeine kodeine merged commit a8d6e37 into kodeine:master Mar 27, 2017
@kodeine
Copy link
Owner

kodeine commented Mar 27, 2017

so its merged on master now, can you please update the docs which reflects the new function names and changes?

@stygiansabyss
Copy link
Contributor Author

I think I updated all of them. If I missed something, let me know and I'll hop on it.

@eumanito
Copy link

Hi @kodeine, the merge is done? The code still contains is() and can().

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.

None yet

3 participants