Skip to content

Conversation

zaerl
Copy link
Collaborator

@zaerl zaerl commented Oct 6, 2025

Motivation for the change, related issues

Context: #2686

Documentation should be updated to reflect the new change, where the root folder has been moved from / to /wordpress and not a relative path. So all the old require_once 'wordpress/wp-load' should now be documented as a require_once '/wordpress/wp-load' to be sure not having a warning in the console.

Implementation details

Fixed some comments and documentation

Testing Instructions (or ideally a Blueprint)

Nothing to test in particular, a quick check at the code is enough.

@adamziel
Copy link
Collaborator

adamziel commented Oct 6, 2025

Let's spell out what this PR is doing in the title and in the description, it will help the future us browse the commit history without additional clicks

@zaerl zaerl changed the title Fix #2686 Fixing wrong documentation regarding PHP code examples Oct 6, 2025
@zaerl zaerl changed the title Fixing wrong documentation regarding PHP code examples Fixing old documentation regarding PHP code examples Oct 6, 2025
@zaerl
Copy link
Collaborator Author

zaerl commented Oct 6, 2025

Let's spell out what this PR is doing in the title and in the description, it will help the future us browse the commit history without additional clicks

Done, thanks.

@adamziel adamziel merged commit 2b459e1 into trunk Oct 6, 2025
23 checks passed
@adamziel adamziel deleted the fix/issue-2686 branch October 6, 2025 09:47
@adamziel
Copy link
Collaborator

adamziel commented Oct 6, 2025

Thank you!

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.

2 participants