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

Missing dashboard message #2714

Merged
merged 5 commits into from
Jun 18, 2024
Merged

Missing dashboard message #2714

merged 5 commits into from
Jun 18, 2024

Conversation

tomjn
Copy link
Member

@tomjn tomjn commented Jun 18, 2024

If a user deletes the entire www folder and the dashboard with it there'll be no www/default/index.php loader file and Nginx will show a forbidden HTTP error when visiting vvv.test.

Instead it now shows this:

Screenshot 2024-06-18 at 15 18 03

Making it clear what happened and suggesting a solution.

Checks

  • I've updated the changelog.
  • I've tested this PR
  • This PR is for the develop branch not the stable branch.
  • This PR is complete and ready for review.

Copy link

update-docs bot commented Jun 18, 2024

Thanks for opening this pull request! Make sure CHANGELOG.md gets updated with this change, additionally any docs that need updated can be found at https://github.com/Varying-Vagrant-Vagrants/varyingvagrantvagrants.org

GitHub
The VVV docs and website. Contribute to Varying-Vagrant-Vagrants/varyingvagrantvagrants.org development by creating an account on GitHub.

@tomjn tomjn modified the milestones: 3.14, 3.13 Jun 18, 2024
@tomjn tomjn marked this pull request as ready for review June 18, 2024 14:25
@tomjn tomjn requested a review from a team June 18, 2024 14:25
@tomjn
Copy link
Member Author

tomjn commented Jun 18, 2024

I was able to test this by renaming index.php and visiting the dashboard

@tomjn tomjn merged commit 9292394 into develop Jun 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant