Opened 11 years ago
Last modified 5 years ago
#22016 new New feature
Automatically reload i18n files on change, when DEBUG is True
Reported by: | Patrick Michaud | Owned by: | Patrick Michaud |
---|---|---|---|
Component: | Internationalization | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | m17.admin@… | Triage Stage: | Accepted |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | yes |
Easy pickings: | no | UI/UX: | no |
Description
It would make things easier for our text editors if they didn't need to run compilemessages to see their changes on their builds. This would need to cover python translation, templates, and the javascript catalog. This would create parity between seeing text changes and seeing python changes.
This is similar to #9523, only I'd like to have a check for compilation when translations are used.
I have a patch that has unit tests, and implements changes in trans_real and in the javascript_catalog. I'll send a pull request after my CCLA is sent in.
Change History (5)
comment:1 by , 11 years ago
comment:2 by , 11 years ago
Triage Stage: | Unreviewed → Accepted |
---|---|
Version: | → master |
I'm not very familiar with translations, but this seems like a reasonable request.
comment:3 by , 10 years ago
Patch needs improvement: | set |
---|
you forget to link the PR: https://github.com/django/django/pull/2421
Currently your patch needs to be updated, could you do that?
Also it would be great if you add some small docstrings to your tests.
comment:4 by , 9 years ago
Cc: | added |
---|
comment:5 by , 5 years ago
With the current code base, this can be achieved by adding the .po suffix for watching in https://github.com/django/django/blob/master/django/utils/translation/reloader.py, and then compile it.
However, the .po suffix seems to been explicitly left out, perhaps there is not a consensus in this behavior for every workflow.
This seems straightforward to implement at this moment, can someone confirm this?
Branch is at https://github.com/vegitron/django/tree/ticket_22016