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

Only register the telescope service provider if its enabled #242

Merged

Conversation

zach2825
Copy link
Collaborator

I compiled and included telescope assets since they are tiny.
Also disabled telescope by default.
If you want to use it,
you can set `TELESCOPE_ENABLED=true` in your .env file
@zach2825
Copy link
Collaborator Author

@allella This should resolve the composer issue you saw when telescope is not enabled. I only register the telescope provider if its enabled

@allella allella merged commit 5ad9f6c into hackgvl:develop Mar 26, 2024
1 check passed
@allella
Copy link
Member

allella commented Mar 26, 2024

@zach2825 this was merged and pulled, but I'm still getting the error on stage.

Would the following in config.app be causing the class to load?

App\Providers\TelescopeServiceProvider::class

@allella
Copy link
Member

allella commented Mar 26, 2024

Sorry, I meant to say config/app.php

I just patched that file on stage, removing the App\Providers\TelescopeServiceProvider::class and that did allow composer to run without an error.

allella added a commit to allella/hackgreenville-com that referenced this pull request Mar 27, 2024
…ically register in app/Providers/AppServiceProvider.php if Telescope is enabled in the .env environment variables. Fixes hackgvl#242.
allella added a commit that referenced this pull request Apr 3, 2024
…ically register in app/Providers/AppServiceProvider.php if Telescope is enabled in the .env environment variables. Fixes #242. (#243)
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.

Disable telescope by default
2 participants