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

Try implementing Django's JavaScript translation catalog again #1434

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

courtneycb
Copy link
Contributor

In #1415 an attempt was made at using Django's JavaScript translation catalog. However an error occurred (detailed in #1409) and the changes had to be rolled back due to a survey running on the develop site at the time.

This PR implements the same changes that were in #1415 but also has the changes to create the djangojs message file.

Hopefully resolves #1409

@courtneycb courtneycb added backend Related to the software powering the website internationalization Related to translating the website labels Sep 8, 2020
@courtneycb courtneycb self-assigned this Sep 8, 2020
@eAlasdair eAlasdair mentioned this pull request Sep 25, 2020
Copy link
Contributor

@eAlasdair eAlasdair left a comment

Choose a reason for hiding this comment

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

It looks fine but I also missed the first bug -_-

Think about whether or not this should go into the current release though

@courtneycb
Copy link
Contributor Author

It looks fine but I also missed the first bug -_-

Think about whether or not this should go into the current release though

Not putting this into the current release

@eAlasdair
Copy link
Contributor

Since this also adds the gettexts, this PR should implement them into everything added in the current release once it's out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to the software powering the website internationalization Related to translating the website
Development

Successfully merging this pull request may close these issues.

Use Django's JavaScript Catalog
2 participants