FeatureRequest: course specific lang related improvement
- Dominant language
- Python
- Stars
- 435
- Forks
- 124
- Avg merge
- 5h 40m
- Merged PRs (30d)
- 14
Description
1. As of dc9c2f7fdf297e82f2167d198bba113b69afa703, feedback locale is configured to `settings.RELATE_ADMIN_EMAIL_LOCALE`. That's weird when `course.force_lang` is not None (i.e., not determined by browser).
2. When `course.force_lang` is not None, email locale should be `course.force_lang` instead of `settings.RELATE_ADMIN_EMAIL_LOCALE` (implemented in 7061164d951cecc9775dc68cfd15b45c1fe7a9af).
3. Ideally, add a field for `Course` named `prefered_email_locale` default to `settings.RELATE_ADMIN_EMAIL_LOCALE`, and can be None (which hehave like 7061164d951cecc9775dc68cfd15b45c1fe7a9af was not implemented). `prefered_email_locale` will be overriden by `course.force_lang` when `course.force_lang` is not None. This is a little complicated and will result in more email activities for emails which was previously cc to staff while sending to students (because there will be no cc any more in cases when a student receive email with locale determined by their browser, while staff receive email with locale determined by `prefered_email_locale`).
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by tracing the feedback-email locale handling and Course language fields, comparing commits dc9c2f7fdf297e82f2167d198bba113b69afa703 and 7061164d951cecc9775dc68cfd15b45c1fe7a9af. Done means defining the course email-locale preference, applying course.force_lang when present, and preserving the intended student and staff email behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- django, python
- Domain
- backend, internationalization, localization
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100