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

11.x Adds “Storage Linked” to the artisan about command #53656

Closed
wants to merge 3 commits into from

Conversation

adampatterson
Copy link
Contributor

I've added a "Storage linked" item to the Environment section of the about command.

While helping some developers to debug why their media isn't displaying correctly, it is often a result of the storage folder not being linked.

I thought adding simple item to the about command would be a helpful addition.

Running php artisan about will show an item labeled "Storage Linked" with a value of LINKED or NOT LINKED.

@ahinkle
Copy link
Contributor

ahinkle commented Nov 24, 2024

Cool idea but the storage location can be configured in the filesystems configuration file and can contain multiple symbolic links.

This PR is suggesting that we are trusting that it's not changed from the default configuration.

@adampatterson
Copy link
Contributor Author

@ahinkle I could check to see if the default is local and then confirm that all paths exist.

@taylorotwell
Copy link
Member

See above. 👍 Not sure what best solution would be here... maybe a list of links - not really sure.

@adampatterson
Copy link
Contributor Author

Thanks @taylorotwell, I'm working on an all paths if the file driver is local, then I'll show a new section and the status of each link.

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.

3 participants