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

Seedlet: Fix a fatal error in Gutenberg #7940

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Seedlet: Fix a fatal error in Gutenberg #7940

merged 1 commit into from
Jul 8, 2024

Conversation

dsas
Copy link
Contributor

@dsas dsas commented Jul 8, 2024

4bf4fb9 introduced a bug when including wpcom-colors.php by using the editor. It was missing a directory separate and so it fataled with an error message like:
`PHP Fatal error: Uncaught Error: Failed opening required '/xxx/seedlet/incwpcom-colors.php' (include_path='.') in /xxx/seedlet/inc/wpcom-editor-colors.php:3

Changes proposed in this Pull Request:

Related issue(s):

#7937

4bf4fb9 introduced a bug when including `wpcom-colors.php` by using the
editor. It was missing a directory separate and so  it fataled with an
error message like:
`PHP Fatal error:  Uncaught Error: Failed opening required '/xxx/seedlet/incwpcom-colors.php' (include_path='.') in /xxx/seedlet/inc/wpcom-editor-colors.php:3
@dsas dsas self-assigned this Jul 8, 2024
Copy link
Contributor

github-actions bot commented Jul 8, 2024

Preview changes

I've detected changes to the following themes in this PR: Seedlet.

You can preview these changes by following the links below:

I will update this comment with the latest preview links as you push more changes to this PR.
⚠️ Note: The preview sites are created using WordPress Playground. You can add content, edit settings, and test the themes as you would on a real site, but please note that changes are not saved between sessions.

@dsas dsas merged commit 2bcca3f into trunk Jul 8, 2024
3 checks passed
@dsas dsas deleted the fix/seedlet-fatal branch July 8, 2024 15:45
@mrfoxtalbot
Copy link

@dsas, I am still seeing this error in production. Should it be working by now? Thank you!

@dsas
Copy link
Contributor Author

dsas commented Jul 8, 2024

@mrfoxtalbot bear with me, I'm having a deployment nightmare

@dsas
Copy link
Contributor Author

dsas commented Jul 8, 2024

@mrfoxtalbot ok, I've deployed Seedlet.

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.

2 participants