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.

Sign up for GitHub

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

Jump to bottom

Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory #71

Closed
leeqvip opened this issue Jun 19, 2024 · 0 comments
Closed

Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory #71

leeqvip opened this issue Jun 19, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@leeqvip
Copy link
Member

leeqvip commented Jun 19, 2024

ModelLoaderFactory was added in #69.
It is more expressive and elegant to use Laravel's built-in factory class.
https://github.com/laravel/framework/blob/11.x/src/Illuminate/Support/Manager.php
@leeqvip leeqvip added the enhancement New feature or request label Jun 19, 2024
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jun 30, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
7f86d1f 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jun 30, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
cb8bc81 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jun 30, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
8ac214e 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jun 30, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
e669afa 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jun 30, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
41c5647 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jul 1, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
6027624 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jul 1, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
0b14d17 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jul 1, 2024
@Dobmod
feat: use Laravel's built-in Manager class and integrate authorizatio…
320a77c 
…n Gates

- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
- Integrate Laravel's built-in authorization Gates (php-casbin#70)
@Dobmod Dobmod mentioned this issue Jul 1, 2024
Merged
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jul 5, 2024
@Dobmod
feat: use Laravel's built-in Manager class
62f8de5 
- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
Dobmod added a commit to Dobmod/laravel-authz that referenced this issue Jul 5, 2024
@Dobmod
feat: use Laravel's built-in Manager class
fe5839f 
- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (php-casbin#71)
leeqvip pushed a commit that referenced this issue Jul 5, 2024
@Dobmod
feat: use Laravel's built-in Manager class (#72)
4d49aef 
- Use Laravel's built-in abstract Manager class instead of ModelLoaderFactory (#71)
@leeqvip leeqvip closed this as completed Jul 5, 2024
@leeqvip leeqvip reopened this Jul 5, 2024
@leeqvip leeqvip closed this as completed Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@leeqvip