-
Notifications
You must be signed in to change notification settings - Fork 69
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
Update CSV export language codes to align with the languages supported by the server #8190
Conversation
Test the buildOption 1. Jetpack Beta
Option 2. Jurassic Ninja - available for logged-in A12s🚀 Launch a JN site with this branch 🚀 ℹ️ Install this Tampermonkey script to get more options. Build info:
Note: the build is updated when a new commit is pushed to this PR. |
Size Change: 0 B Total Size: 1.26 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this quick fix. The code looks good. Test cases 1 and 2 passed for me. Test case 3 didn't fully pass for me. Locale query string param was es
, but the CSV is not in Spanish.
Thank you for the quick review! The behavior you're experiencing is expected because the Spanish translation files are not present in your environment, they are only available in production. I've made the necessary updates to the steps. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update and the explanation. All three test cases are passing for me.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ LGTM
…d by the server (#8190) Co-authored-by: Dan Paun <[email protected]>
Changes proposed in this Pull Request
At present, the default WordPress locale codes do not align with the supported languages on the server. This PR modifies the WordPress locale codes to match the supported languages.
Additionally, it hides the CSV export modal if the language is not supported.
Testing instructions
Case 1
Settings
->General
.Spanish
.Payments
->Settings
.Case 2
Settings
->General
.Bulgarian
.Payments
->Settings
.Case 3
Settings
->General
.Spanish
.Payments
->Transactions
.Remember the language
settings checkbox.Site language
->Spanish
and click download.locale
param in the request ises
.npm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge