-
Notifications
You must be signed in to change notification settings - Fork 344
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 i18n tags to all Django templates #876
Comments
Hello @brylie, |
@mharyam yes please :-) I'll assign you to the issue. Please let us know how we can be of assistance. |
Thank you, I will. |
Hello @brylie, For this task, do you think it is ideal I add template tag {%trans%} to all the individual templates? |
@mharyam, yes. Follow the guidelines from Django documentation. |
Thank you so much @mharyam! 😃 I've reviewed your pull request and made some minor suggestions with code changes that you can automatically accept. |
Hello @brylie Thanks a lot for taking out time to review! I have accepted all the suggestions. It looks like you still need to merge the pull request, so it can be updated to the main branch Thank you. |
Thank you @brylie for taking out time to approve the pull request. Do you think there are other pages I need to add translation to? |
Basically, just look for a |
Hi @brylie, I'd like to help with this issue. Please let me know if there is any work that still needs to be done for this issue. |
As we port templates from the SPA app over to regular Django templates, ensure that we add localization support for all static texts. Prior to the 2.0 release, review all templates to make sure the internationalization is complete.
The text was updated successfully, but these errors were encountered: