Skip to content

Conversation

fsbraun
Copy link
Member

@fsbraun fsbraun commented Jan 10, 2023

Description

This PR adds localization and transifex support.

  • Transifex config is found in .tx/config
  • German and English localization files are added (more to come from transifex)
  • Quotes in three templates adjusted so that ./manage.py makemessages --all creates the correct strings

Related resources

  • #...
  • #...

Checklist

  • I have opened this pull request against master
  • I have added or modified the tests when changing logic
  • I have followed the conventional commits guidelines to add meaningful information into the changelog
  • I have read the contribution guidelines and I have joined #workgroup-pr-review on
    Slack to find a “pr review buddy” who is going to review my pull request.

@fsbraun fsbraun requested a review from marksweb January 10, 2023 16:30
@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Merging #305 (648dfd1) into master (f13e299) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #305   +/-   ##
=======================================
  Coverage   89.75%   89.75%           
=======================================
  Files          67       67           
  Lines        2196     2196           
  Branches      293      293           
=======================================
  Hits         1971     1971           
  Misses        170      170           
  Partials       55       55           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@@ -19,7 +19,7 @@ <h4>{% blocktrans %} Version number: {{ version_number }}{% endblocktrans %}</h4
{% csrf_token %}
<input class="button confirm-link js-versioning-keep-sideframe"
type="submit"
value="{% trans 'Yes, I\'m sure' %}">
value="{% trans "Yes, I\'m sure" %}">
Copy link
Member

@marksweb marksweb Jan 11, 2023

Choose a reason for hiding this comment

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

We might be better off using translate as the tagname.

trans still exists as an alias in django main, but I think it'll get removed at some point.

You also don't need to escape the inverted comma now that you're double quoting the string.

Copy link
Member Author

Choose a reason for hiding this comment

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

True. I've replaced all {% trans %} tags with {% translate %}

@fsbraun fsbraun merged commit e880798 into django-cms:master Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants