pantheon-systems / wordpress-composer-managed

WordPress boilerplate with Composer, easier configuration, and an improved folder structure
https://docs.pantheon.io/guides/wordpress-composer
MIT License
4 stars 4 forks source link

[CMSP-1140] fix subdirectory paths #142

Closed jazzsequence closed 3 months ago

jazzsequence commented 3 months ago

This pull request removes the unnecessary code for handling wp-admin URLs. The code was not working as intended and caused issues.

This has been tested on a subdirectory multisite (which was the only thing that was broken or affected). Subdomain multisite should be unaffected (there are conditionals specifically targeting subdirectory multisite) but will be tested as well assuming automated tests pass.

The primary issue was admin URLs which are not (currently) tested with automated testing.

blocked by #140

jazzsequence commented 3 months ago

Ohhhh, the robot should not change composer files....

jazzsequence commented 3 months ago

not sure why this is pulling the full history. closing and reopening a clean pr