-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Include translation files for dataTables #303
base: master
Are you sure you want to change the base?
Conversation
Tests need fixing. Will do so later |
@phw Could you please provide insight into why the tests for this PR have been failing? As far as I understand, this PR simply adds internationalization (i18n) to the DataTables, as previously implemented (refer to #298). Is the issue with the test file, as that is what i could infer from the following screenshot. |
@navi-prem It fails because the picard-website/website/frontend/babel.py Lines 42 to 46 in df7b35f
I haven't yet checked how we can replicate this in the tests. If you want to help you are very welcomed to investigate. Below is the relevant test output
|
Summary
Problem
This adds translations for the dataTable component added to the plugins page in #298 by @navi-prem . See https://datatables.net/plug-ins/i18n/