-
Notifications
You must be signed in to change notification settings - Fork 65
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
Latest Site Hint: Don't hint when the year is the same. #1239
Conversation
…as the current site, that has an identifier.
Just noting; I didn't get a chance to add a unit test today, I'll follow up with one next week |
Github actions are completely broken due to unrelated changes, and I can't run unit tests locally due to incompatibilities in how this codebase is setup. I can't duplicate the failing unit test outside of the unit tests. |
Fix PHPCS feedback.
I tried to fix the tests, but it's going to be stuck between a unit test fail and a static analysis fail atm! |
Fix latest static analysis - forgot this was why!
Having trouble testing this PR - both locally and on a sandbox - will continue looking later. |
To avoid php warning
Testing the PR works, just can't figure out why the unit tests are failing |
Did this another way using a toggle, since there appears to be a few edges cases (probably more too). |
I've created an additional site for Canada 2024, such that they can have an English and French site. See https://wordpress.slack.com/archives/C08M59V3P/p1706836174954409
However, this has caused the latest site hint to suggest this:
This change updates the code to strip the
-identifier
part of/2024-fr/
from the url for the comparison.How to test the changes in this Pull Request: