Protoqol / Prequel

Prequel for Laravel. Clear and concise database management. Unfortunately, not actively maintained due to time constraints.
https://packagist.org/packages/protoqol/prequel
MIT License
1.54k stars 96 forks source link

[FEATURE REQUEST] Install only in dev env #16

Closed wrtisans closed 5 years ago

wrtisans commented 5 years ago

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

it would be great install it only on dev environent, if its possible now and im missing someting please let me know

Describe the solution you'd like A clear and concise description of what you want to happen.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

QuintenJustus commented 5 years ago

Hey, I’m not really sure what you mean? Could you fill in the other fields aswell, and be more clear on what exactly you’d like to see? Thanks 😁

aaronsaray commented 5 years ago

I think what @wrtisans is saying is that they'd only like this installed in dev environment. In the Authorised middleware, there is a check to make sure that the functionality is not enabled in production. But, regardless, the service provider and other checks are still executed. I think the request has to do with not auto-registering the service provider and following the example in barryvdh/laravel-ide-helper where user registers it by hand. One could argue that not registering the service provider is a micro-optimization, and for ease of use, just allow it to continue registering automatically. Others might say they don't want non-production things even installed on production.

@wrtisans you can get your required desire, I think, by following the above example and adding this package to your dont-discover section of your composer.json file.

wrtisans commented 5 years ago

@aaronsaray great! thats what i was looking!!!