Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

[V5] Support for custom key names on Role,Permission #1913

Merged
merged 1 commit into from
Nov 17, 2021
Merged

[V5] Support for custom key names on Role,Permission #1913

merged 1 commit into from
Nov 17, 2021
+41 −27

Conversation

erikn69
Copy link
Contributor

@erikn69 erikn69 commented Nov 4, 2021 •
edited
Loading

#1909 (comment)

Could you please break this PR up in smaller ones that are easier to review?

First part only changes propertie id to getKey() or getKeyName()

  • Support for custom key names on Role,Permission model instead of defaut one.

@freekmurze freekmurze merged commit fb9e680 into spatie:main Nov 17, 2021
@freekmurze
Copy link
Member

Thanks!

@erikn69 erikn69 deleted the small_part_1 branch February 17, 2023 14:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@erikn69 @freekmurze