Closed Bug 823536 Opened 12 years ago Closed 11 years ago

Form on localized Get Involved page not sending an auto-response

Categories

(www.mozilla.org :: Bedrock, defect)

Product:
www.mozilla.org
Component:
Bedrock
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: davidwboswell, Unassigned)

References

(
URL
)

Details

(Whiteboard: u=user c=l10n p=) 

Pascal just pushed the French Get Involved page to production and I was testing the form on the page.  

When I submitted an inquiry it went through, although I was expecting to get an auto-response but didn't.

For reference, I submitted the form and selected 'Autre' as the area of interest.  When selecting 'Other' in the English form, I get an auto-response.

I haven't tested the other areas of interest on that page yet, but can.

There was some earlier work done to get the form ready for localization that may be relevant:

https://bugzilla.mozilla.org/show_bug.cgi?id=756093
talking with Rik today, it was working during our initial implementation of the page but that was a few months back. The automatic test for auto-responses may not be activated/testing the right thing/testing locales/needs update.
moving to bedrock as this is more of a code issue than l10n
Component: L10N → Bedrock
Is anybody working on this?

I can try to download bedrock to test locally, but time ago I was told there were some incompatibilities that were not allowing people to make it work locally.
(In reply to Rubén Martín [:Nukeador] from comment #3)
> Is anybody working on this?

I don't believe anyone is working on this now.

For background, Giorgos had this working at some point over the Summer, but something must have changed in the meantime.  He could be a good person to talk to.
 
> I can try to download bedrock to test locally, but time ago I was told there
> were some incompatibilities that were not allowing people to make it work
> locally.

Here's some instructions about getting Bedrock running locally.  

http://bedrock.readthedocs.org/en/latest/install.html

If they don't work, we should open a separate bug to get the instructions updated.
Adding Ben to the bug for his feedback.

Ben, this bug is blocking our ability to roll out and promote localized versions of the Get Involved page.  Do you have a sense of the best way to get it fixed?  Should we go through the webprod process or would it be quicker if Mozilla Hispano or another volunteer took this?
Whiteboard: u=user c=l10n p=
Based on the above it looks like a regression so probably quickest to have Giorgos look. It might be a very quick fix.

It looks like jbertsch added whiteboard tags to this one but it's currently unassigned.

Giorgos, can you take a look at the above bug and let us know if this is something you have time to look into in the next week or so, or if we should look for other resources.
Flags: needinfo?(giorgos)
Looks like the deal is that the auto-responder function can't find the localized template for the email. It's looking for these email templates in a location like:

    bedrock/locale/{LOCALE}/templates/mozorg/emails/{INTEREST_ID}.txt

For example:

    bedrock/locale/fr/templates/mozorg/emails/other.txt

These templates appear however to live at (for example):

    bedrock/locale/fr/emails/other.txt

Several locales seem to follow the same pattern as the fr locale. The only one following the expected pattern at present is "sr". While bs, de, en-GB, es-ES, fr, he, hi-IN, hr, lt, nl, pt-BR, ro, sq, sr, sw, tr, zh-CN, and zh-TW all have it the other way, which is to say, the way that's broken. Notice "sr" is in both lists, as that locale has what appear to be valid templates in both places.

Pascalc: How would you like to proceed? I can change the function to expect the templates at "bedrock/locale/{LOCALE}/emails/{INTEREST_ID}.txt", or I can leave it as is and you can move all the templates to the place where the code is looking.
Also note that a cursory glance at a couple of the fr email templates would suggest that few of them are actually translated.

http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/locales/fr/emails/
Thanks pmac, I can move the folders on the svn side tomorrow (and as for the French ones, we are currently rewriting them so since the feature wasn't working, I didn't commit the initial translation).
Great! Thanks :pascalc!
Flags: needinfo?(giorgos)
I tried with es-ES and it works, I will move the other locales tomorrow and close this bug. Thanks
Fantastic! I'll probably end up submitting a PR to have it email the site admins when it can't find a template, just to warn us that someone didn't get an email, and we need to file a bug.
Adding Tim, the point person for the Dutch Get Involved page, to this so he's aware that localized auto-responses to inquiries will be working soon.  All point people for the three live locales (ES, FR and NL) are now on this bug.
Pascal, I've just tried and it's not working yet for es-ES on production.
I've checked again, and no auto-response is sent for es-ES.

The problem is that we have to handle more than 100 emails per week manually, which is time very consuming.

pmac, what can we do to help you fix this asap?

Thanks!
Flags: needinfo?(pmac)
I just tested it with the email address "pmclanahan+es-contribute-test@gmail.com" and it worked perfectly. I selected webdev as my interest group. What interest did you select when testing? Perhaps it's an issue with only some of the groups.
Flags: needinfo?(pmac)
FYI, it also worked for me and Inma when we tried 2 weeks ago, I think it's probably a category issue as pmac suggests.
I tried with "Programación" (Coding)
That is the issue then. It appears that some of the categories we have do not have corresponding email templates.

Categories selections:

support
qa
coding
marketing
localization
webdev
addons
design
documentation
education
other
suggestions
issues

Categories for which we have email templates:

addons
design
documentation
education
issue
marketing
other
qa
suggestions
support
webdev
So the ones we're missing are:

coding
issues
localization

It seems that the "issue.txt" file is just missing an "s". But the other two are really missing, and probably are for all locales for which we have templates.
Coding is the one functional area in English that doesn't use an auto-response.  For reference, details about all of the categories on the English Get Involved page (including point person, auto-response or manual response and link to auto-response text) are at

https://wiki.mozilla.org/Contribute/Funnels#By_Functional_Area

What we can do here is get the template text Josh Matthews uses in his manual responses and make that available for you to localize.

For localization, there is an auto-response available to localize.  This was added relatively recently though, so it's possible it wasn't picked up in the set of strings.  It is at

https://github.com/mozilla/bedrock/blob/master/apps/mozorg/templates/mozorg/emails/localization.txt
The latest text Josh is using as a template for Coding responses is now at

https://wiki.mozilla.org/Contribute/Canned_responses#Coding
In Spanish we use similar templates for addons, webdev and coding, because all of that goes to the same functional area, Labs.

Inma, just copy that for coding.txt and add a new one for localization.txt and we should be done.
It sounds like we can close this now.  If any other locales don't have functioning auto-responses it sounds like we know where to look.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Just tested for QA and Support for dutch but no auto responce
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Tim, you need a templates/mozorg/email folder with the autoresponses:

https://svn.mozilla.org/projects/mozilla.com/trunk/locales/nl/

Check:

https://svn.mozilla.org/projects/mozilla.com/trunk/locales/es-ES/templates/mozorg/emails/

Maybe it's better to open individual bugs for each locale problem.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
(In reply to Rubén Martín [:Nukeador] from comment #26)
> Tim, you need a templates/mozorg/email folder with the autoresponses:
> 
> https://svn.mozilla.org/projects/mozilla.com/trunk/locales/nl/
> 
> Check:
> 
> https://svn.mozilla.org/projects/mozilla.com/trunk/locales/es-ES/templates/
> mozorg/emails/
> 
> Maybe it's better to open individual bugs for each locale problem.

excuse me but nl was one of the locales with translated and actived responces (i am told), so i think this is the right bug!

http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/locales/nl/emails/
I can document this better to make sure people are aware of the right place to put things.  To confirm, the correct location is:

http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/locales/[locale_code]/templates/mozorg/emails/

I'll put this information for now on the Funnels wiki page but we'll be creating a toolkit for funnels soon that packages things up better.

https://wiki.mozilla.org/Contribute/Funnels#By_Locale
(In reply to David Boswell from comment #29)
> I can document this better to make sure people are aware of the right place
> to put things.  To confirm, the correct location is:
> 
> http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/locales/
> [locale_code]/templates/mozorg/emails/

Correct.
(In reply to Pascal Chevrel:pascalc from comment #9)
> Thanks pmac, I can move the folders on the svn side tomorrow (and as for the
> French ones, we are currently rewriting them so since the feature wasn't
> working, I didn't commit the initial translation).

so I thought it was done. shall I move the files or does pascal?
Tim, please do, I am into other time consuming projects currently and I don't expect to have time to work on the contribute page before the end of Mobile World Congress
Tim,

If you need help moving things around I can do it, just let me know.
(In reply to Brian King [:kinger] from comment #33)
> Tim,
> 
> If you need help moving things around I can do it, just let me know.

thanks, i am working this weekend and have probably time for it tomorrow evening(or later). so if someone can do it would be nice.
I moved the files, i figure it takes a moment before it will work (at this moment not). I will test it again tomorrow
Tim, I just pushed your files to production, it should work in 20mn
Tested and everything works, thanks all
You need to log in before you can comment on or make changes to this bug.