1080066 - Implement Basket Send Email To Steward(s) Based On Interest + Language

Status: RESOLVED FIXED

(Websites :: Basket, defect)

Description

[Jessilyn Davis]

Now that we have ability for basket to send an email to Steward based on Interest (Bug 1066704) - we need the ability to send to a different steward based on Interest + Language.

At present, the need is for 1 email address for all interests in a non-en Locale (ie all ES interests go to one email address, all HU interests go to one email address, vs. EN interests have different email addresses per steward)

In the future, we'll want to have the ability to send to individual stewards based on interest + language - and - the ability to update this info from an admin interface (rather than a code-push).


This blocks the launch of l10n versions of the Get Involved page.

What is the recommended way forward to implement the fastest and keep the launch on track for next week?


Jennie - can you put in the list of locales + the steward's address that should be alerted for that locale here? OR if the stewards are the same for the locales as what is currently in bedrock (https://github.com/mozilla/bedrock/blob/master/bedrock/mozorg/email_contribute.py#L96-L127), please let the team know and they can pull the info from there to use.

Comment 1

[Jessilyn Davis]

Just heard word from Jennie:

The list of locales + contacts here is correct:
https://github.com/mozilla/bedrock/blob/master/bedrock/mozorg/email_contribute.py#L96-L127


Please add this locale + email address combo:

hi-IN = [e-mail]

Comment 2

[jhalperin]

Stewards need to be Changed:

lt = lt 	[e-mail] 
sr = [e-mail]
zh-TW = [e-mail]

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/basket

https://github.com/mozilla/basket/commit/6218efbd11d197f5d0f573d5b1325dc6473e8a22
Fix bug 1080066: Add per-locale stewards and fix email field.

Adds a new model for tracking per-locale stewards for a given interest.

Removes the magic in CommaSeparatedEmailField that converts the value
to a list, as this was screwing up the text inputs in the admin.

Also uses a text input for the fields instead of a textarea.

https://github.com/mozilla/basket/commit/f73924ca2ff244801d3dab42e1722be1fe51e42d
Merge pull request #115 from Osmose/locale-stewards

Fix bug 1080066: Add per-locale stewards and fix email field.