-
Notifications
You must be signed in to change notification settings - Fork 66
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
wordcamp-participation-notifier.php fails to require wporg-mu-plugins/pub/profile-helpers.php #776
Comments
Related #769 |
🤔 , does this line not return early for you? Lines 13 to 20 in 89b2afc
|
@iandunn, I modified those line to the following as a test:
Here is the the output I'm seeing:
My local is running on wordpress.test. |
Ah, that's right. We've got The best fix for this case is probably something like -if ( 'local' === wp_get_environment_type() ) {
+if ( 'local' === WORDCAMP_ENVIRONMENT ) {
return;
} |
@iandunn, the suggested code change above resolved the error I was seeing. Does it makes sense to update the repository with an update to try to correct this for the next person who tries to setup a local environment with Docker? |
Describe the bug
On a new install of the wordcamp.org evironment via Docker, this error is encountered.
This line was lasted edited 19 days ago via 89b2afc
To reproduce
Steps to reproduce the behavior:
docker compose up
command in step 7Expected behavior
See
Workaround
Comment line 29 of wordcamp-participation-notifier.php
WordCamp
local wordcamp.test
System (please complete the following information):
The text was updated successfully, but these errors were encountered: