Skip to content
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

Add an en-GB localization file. #1109

Merged
merged 1 commit into from
Sep 8, 2024

Conversation

drgrice1
Copy link
Member

@drgrice1 drgrice1 commented Sep 1, 2024

This isn't exactly needed, but is added for consistency. As for webwork2, translation isn't really needed in the .po file. However, if someone wants to do so it could be done.

Note that the en.po file was updated as well. This file hasn't been updated for a while since it really doesn't need to be.

This isn't exactly needed, but is added for consistency.  As for
webwork2, translation isn't really needed in the `.po` file.  However,
if someone wants to do so it could be done.

Note that the `en.po` file was updated as well.  This file hasn't been
updated for a while since it really doesn't need to be.
Copy link
Contributor

@somiaj somiaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think there is anything to actually test here, since there are no dates on the pg side of things, and no transitions are being used yet.

@drgrice1
Copy link
Member Author

drgrice1 commented Sep 3, 2024

Yeah, this is just for completeness. Also, I think it would be an error if the file did not exist and the language was selected when a problem renders.

@somiaj
Copy link
Contributor

somiaj commented Sep 3, 2024

I did test a few problems and got no errors, so that at least is good.

@somiaj
Copy link
Contributor

somiaj commented Sep 4, 2024

FWIW, it seems using en-GB in webwork without this applied still works without any error when rendering problems, so looks like maketext just returns the inputted string without any error if given localization which doesn't have a .po file.

@pstaabp
Copy link
Member

pstaabp commented Sep 6, 2024

If anyone else is testing this, you also need to checkout openwebwork/webwork2#2553

Copy link
Member

@pstaabp pstaabp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not much to test here without translations.

@Alex-Jordan Alex-Jordan merged commit a9ba458 into openwebwork:develop Sep 8, 2024
3 checks passed
@drgrice1 drgrice1 deleted the en-GB-localization branch September 10, 2024 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants