-
Notifications
You must be signed in to change notification settings - Fork 82
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
Compatibility with wpml #153
Comments
We have recently had to solve this problem. We found there were 3 steps required to get this working properly:
|
Same issue here (WPML is very popular!). Could you share the exact code to add to functions.php please? It would be incredible helpful for me... |
…_responsive onsomething: Adding Yoast SEO (wordpress-seo) plugin
Plugin version: 1.10.0 I've had a simmilar issue. to actually fix it I've frozen the plugin source for our case and applied this piece of code in
Basically what it does is to not fully delete the requested path if it has a language code. Otherwise the $requested_path for I haven't run any tests for it or thoroughly check other cases but this might be a hint for somebody else. |
@CPROps @jameswburke looks like you may have crafted a downstream fix, mind pushing that upstream here for others to benefit from? |
@jeffpaul - We (CPR) don't have a downstream fork of this repo that I can see, but it's possible that we inherited something along the way. You may want to check with whomever is maintaining AlleyInteractive's fork. |
I can't make this play along with wpml. maybe because wpml creates virtual directories(ex: /fr/ , /pt/ don't really exist on the server) or any other issue ...
But the fact is that my 301 redirects that include one of this virtual directories results in a 404.
I'm using NGINX (powered by trellis and bedrock).
Anyone have a clue?
Thanks in advance
Rui
The text was updated successfully, but these errors were encountered: