Closed
Bug 523403
Opened 15 years ago
Closed 13 years ago
Improve the usability of the Get Involved contact form
Categories
(www.mozilla.org :: General, defect, P2)
www.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
3.11
People
(Reporter: davidwboswell, Assigned: sgarrity)
References
()
Details
(Whiteboard: r94653,94654,94737,94752,94802,94844 b=staging)
Gerv helped me hook up the contact form on the new Get Involved page, but what we have now is just a temporary fix (partly because the script is on Gerv's server and also because we may want a general solution to this problem).
Here are the open issues with the current form:
* The script on Gerv's server is in Perl and I'm not sure if we can just copy that over or if it needs to be converted to PHP. Based on earlier discussions, I think the server might not allow Perl scripts?
* There is an existing PHP email form on the site that was coded for the OneWebDay poster campaign. Perhaps that can be made more general and allow us to hook in any contact form across the site? Even if that PHP isn't reused, it's probably a good idea for us to review it since a contractor did this and was just checked in.
* For the Get Involved script, I'd like to tweak it so that there area of interest shows up in the subject line. This will make it easier for the group of people monitoring the contribute list to identify what they need to respond to.
Comment 1•15 years ago
|
||
(In reply to comment #0)
> * The script on Gerv's server is in Perl and I'm not sure if we can just copy
> that over or if it needs to be converted to PHP. Based on earlier discussions,
> I think the server might not allow Perl scripts?
Correct, it'll need to be in PHP.
> * There is an existing PHP email form on the site that was coded for the
> OneWebDay poster campaign. Perhaps that can be made more general and allow us
> to hook in any contact form across the site? Even if that PHP isn't reused,
> it's probably a good idea for us to review it since a contractor did this and
> was just checked in.
I modified / rewrote it to work with Get Involved. Somebody seems to have deleted it? Any reason why? :(
> * For the Get Involved script, I'd like to tweak it so that there area of
> interest shows up in the subject line. This will make it easier for the group
> of people monitoring the contribute list to identify what they need to respond
> to.
Sure, that's easy.
Comment 2•15 years ago
|
||
(In reply to comment #1)
> I modified / rewrote it to work with Get Involved. Somebody seems to have
> deleted it? Any reason why? :(
You removed it in r53743. Why?
Reporter | ||
Comment 3•15 years ago
|
||
I can certainly undelete it, but I had removed it since I didn't think it was smart to have unused scripts sitting around on the server. If we made a more general fix for having contact forms I also wasn't sure if that's where it would live.
Reporter | ||
Comment 4•15 years ago
|
||
I was testing the form on the Get Involved page today and noticed it failed because of an error on Gerv's site. We're also going to have a snippet in Nov pointing to this page so it would be good to get a permanent and more robust fix in place by then. Reed, any chance you could look at this soon?
Comment 5•15 years ago
|
||
Error now fixed; my website sync operation deleted the script, because when I created it, I did it directly on the server :-((
Having this admined by professionals would indeed be good :-)
Gerv
Reporter | ||
Comment 6•15 years ago
|
||
From experience so far it seems like several people are submitting the form without modifying any of the fields. It would be helpful to have an email sent only if at least one of the fields is modified. Having some error checking also show up on the page would be nice, but not crucial.
Comment 7•15 years ago
|
||
That sounds like something that's easiest to fix using JS on the page :-)
Gerv
Reporter | ||
Comment 8•15 years ago
|
||
Bumping to p1 in advance of community group meeting.
Priority: -- → P1
Updated•15 years ago
|
Assignee: nobody → reed
Reporter | ||
Comment 10•15 years ago
|
||
Making a note about desired functionality -- doesn't need to be there initially but is a nice to have. It would be good to be able to CC people based on the Area of Interest selected on the form. There are some people interested in getting these inquiries but don't want to sign up for another mailing list so this could be one way to deal with that instead of having someone manually forwarding things on for certain inquiries.
This list of features in the bug is getting a bit long. I should probably write all this down on the wiki somewhere...
Reporter | ||
Comment 11•15 years ago
|
||
List of all form features at
https://wiki.mozilla.org/Mozilla.org/Contribute#Form
Reporter | ||
Comment 12•15 years ago
|
||
FYI, the Get Involved snippet went live yesterday and we've seen a large increase in responses from the form. Bumping importance.
I updated the list of features on the wiki to reflect a couple of small changes:
* Change of subject line to 'Inquiry about [area of interest]' (I realize the previous subject line was going to cause conflicts with how we've been handling things so far).
* A note about stats tracking (like the ability to CC people mentioned in comment #10 this is a nice to have and not needed right away).
Severity: normal → major
Reporter | ||
Comment 13•15 years ago
|
||
One small change to the above information:
* I think the subject line should be 'Inquiry about Mozilla [area of interest]' instead of 'Inquiry about [area of interest]'. When we respond to people it seems more likely that people will recognize this as a response from Mozilla if it says Mozilla in the subject.
Comment 14•15 years ago
|
||
fixing this would also allow faster responses since now david need to forward most of the mails :)
Updated•15 years ago
|
Severity: major → blocker
OS: Mac OS X → All
Hardware: x86 → All
Comment 15•15 years ago
|
||
From the wiki, "Nice to have: For certain areas of interest, we should be able to CC other email addresses for people who want to see the inquiries but don't want to subscribe to the list."
Can I get a list of what e-mail addresses map to what areas? (send OOB, please)
Reporter | ||
Comment 16•15 years ago
|
||
List of emails to CC just sent in an email.
Comment 17•15 years ago
|
||
ok, working fix added in r56984. still want to do a few things like filtering and anti-robot stuff, but this works for now.
Status: NEW → ASSIGNED
Reporter | ||
Updated•15 years ago
|
Severity: blocker → normal
Priority: P1 → P2
Reporter | ||
Comment 18•14 years ago
|
||
Instead of tracking all desired form enhancements in this bug, we should open bugs for each specific feature and link them to this tracking bug. See bug 567823 for an example.
Reporter | ||
Updated•13 years ago
|
Updated•13 years ago
|
Target Milestone: --- → 3.4
Updated•13 years ago
|
Target Milestone: 3.4 → 3.7
Comment 19•13 years ago
|
||
Assigning over to silver orange.
David, are the requirements up to date and is this where they live?:
https://wiki.mozilla.org/Mozilla.org/Contribute/Get_Involved_Evolved
Assignee: reed → steven
Reporter | ||
Comment 20•13 years ago
|
||
(In reply to Mike Alexis [:malexis] from comment #19)
> David, are the requirements up to date and is this where they live?:
>
> https://wiki.mozilla.org/Mozilla.org/Contribute/Get_Involved_Evolved
That wiki page has some high level plans for things to try with the pages. The better place to find requirements are the bugs set as blockers to this bug. Some of those are straight-forward, but some need some additional detail.
Steven, it may be easier to talk through this and then I can follow up with additional notes to clarify things. If that sounds like it would work for you, let me know when works well for you this week. Thanks.
Updated•13 years ago
|
Assignee: steven → nobody
Component: www.mozilla.org → www.mozilla.com
QA Contact: www-mozilla-org → www-mozilla-com
Target Milestone: 3.7 → 3.8
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → steven
Reporter | ||
Comment 21•13 years ago
|
||
FYI, Steven, one thing to keep in mind is that the staging version of this form is not in sync with the production version (see bug 681792 for an example of some confusion this causes).
This probably isn't a surprise to you since you've dealt with the www.mozilla.org staging weirdness before, but wanted to make a note to make sure to grab the latest version of these files from production if you're going to try anything out on stage.
Assignee | ||
Updated•13 years ago
|
Target Milestone: 3.8 → 3.9
Assignee | ||
Comment 22•13 years ago
|
||
If we're sending an auto-responder email for the address submitted with this form, we should require some type of robot prevention (sadly, probably a captcha). The narrowest recaptcha image is 300 pixels wide, though - quite a bit wider than this form.
Is it critical that the form be in the sidebar of these pages, or could it have it's own page (with links from the sidebar)? If it does stay in the narrow sidebar, we'll have to slide the bar open wider or open a lightbox to present the captcha.
Assignee | ||
Comment 23•13 years ago
|
||
Also, is the text for such a default auto-responder (for all Interests without their own custom autoresponder text) written? If not, who should be working on that?
Reporter | ||
Comment 24•13 years ago
|
||
I just talked with Steven on IRC, but wanted to track things here for reference.
There is text for auto-responses on the wiki at
https://wiki.mozilla.org/Mozilla.org/Contribute/Canned_responses
These are blocks of content for different functional areas that people are currently copying and pasting. These will need some tweaking at some point, but is a good place to start.
For the captcha, from the conversation with Steven it sounds like it may not be necessary. I'm happy to discuss more as needed.
Assignee | ||
Comment 25•13 years ago
|
||
David, can you help us correlate the sections inhttps://wiki.mozilla.org/Mozilla.org/Contribute/Canned_responses with the Interests in the flydown - there are duplicates, etc.
Comment 26•13 years ago
|
||
For the coding interest, the response should be
Hi,
I'm Josh, one of the C++ developers on the Mobile team. Glad to hear from you!
To get started with the codebase, have a look at http://developer.mozilla.org/En/Introduction.
If you need any help, you can obtain it on IRC. I recommend visiting #introduction, which is specifically for newcomers. See http://irc.mozilla.org for how to access it.
Feel free to contact me any time if you're having any problems! I'm jdm on IRC, or just josh@joshmatthews.net.
Happy hacking,
Josh
Is it possible to set the reply-to field to go to my email address, as well?
Reporter | ||
Comment 27•13 years ago
|
||
Steven, sure, happy to correlate. For the following, the drop down item is on the left and the corresponding canned response is on the right.
* Helping Users / Support
* Localization / Localization (the first response)
* Testing and QA / Testing
* Coding / see comment #26
* Add-ons / Add-ons
* Marketing / Marketing Test
* Student Reps / Student Reps
* Developer Documentation / Documentation
* User Research / User Research
* Thunderbird / Thunderbird
* Accessibility / Accessibility
* Firefox Suggestions / Firefox Suggestions
* Other / General Test
I don't remember if we talked through this before, but if an area preferred to not have an auto-response, can we have the system not send something if there is no autoresponse text? Although each area has text now, this is something that people will want to try different things with to see what works best.
Reporter | ||
Comment 28•13 years ago
|
||
One other thing that we'll need to provide is the relevant reply-to email address for each auto-response since the person who should get a response will change depending on the area of interest. I'll write that up and post to sgarrity out of this bug so the email addresses don't attract spam.
Comment 29•13 years ago
|
||
Actually, if there's a choice of autoresponding, I would still prefer to manually respond to Coding inquiries instead.
Comment 30•13 years ago
|
||
Form updated committed to staging branch in r94654
Whiteboard: r94653,94654 b=staging
Comment 31•13 years ago
|
||
Josh, I left the autoresponder off for 'Coding'.
Assignee | ||
Updated•13 years ago
|
Whiteboard: r94653,94654 b=staging → r94653,94654,94737 b=staging
Assignee | ||
Comment 32•13 years ago
|
||
As of Mike's recent commits in /branches/staging/:
* form code is cleaned up
* has proper required-field behaviour (HTML5 tags and fallback in-line errors for other browsers)
* email auto-responder has been added with custom emails for particular Interests. See this file for the implemented details: http://viewvc.svn.mozilla.org/vc/projects/mozilla.org/branches/staging/includes/forms/ContributeForm.php?view=markup
This is ready for testing/review on the staging branch: http://www-stage.mozilla.org/contribute/
Reporter | ||
Comment 33•13 years ago
|
||
> * email auto-responder has been added with custom emails for particular Interests.
I just tried this out for the 'Other' option since that goes to me and it worked fine. I got the email and the reply-to was set correctly. I'll reach out to the other people fielding these emails for them to test too.
To clarify here, if an area, like Coding, doesn't want an auto-response it looks like we just don't include that in the function getAutoResponseMessage section?
> * has proper required-field behaviour (HTML5 tags and fallback in-line errors for
> other browsers)
I'm not seeing in-line errors on staging. For instance, if I don't add anything in the 'area of interest' field I'm taken to a blank page with this text: 'Not enough info; all fields must be filled out'.
For the newsletter sign-up, is there any additional information you need from me for that? Perhaps text for the opt-in option?
Comment 34•13 years ago
|
||
(In reply to David Boswell from comment #33)
> > * email auto-responder has been added with custom emails for particular Interests.
>
> I just tried this out for the 'Other' option since that goes to me and it
> worked fine. I got the email and the reply-to was set correctly. I'll
> reach out to the other people fielding these emails for them to test too.
>
> To clarify here, if an area, like Coding, doesn't want an auto-response it
> looks like we just don't include that in the function getAutoResponseMessage
> section?
>
Yep. The same goes for the getAutoResponseReplyTo method.
> > * has proper required-field behaviour (HTML5 tags and fallback in-line errors for
> > other browsers)
>
> I'm not seeing in-line errors on staging. For instance, if I don't add
> anything in the 'area of interest' field I'm taken to a blank page with this
> text: 'Not enough info; all fields must be filled out'.
>
What URL are you testing at? It sounds like it was not completely updated with the changes in r94654. We're using http://mozilla-org-stage.silverorange.com/contribute/.
Reporter | ||
Comment 35•13 years ago
|
||
> What URL are you testing at?
Ah. I was using:
http://www-stage.mozilla.org/contribute/
What's the login information for this staging site?
http://mozilla-org-stage.silverorange.com/contribute/
Comment 36•13 years ago
|
||
(In reply to David Boswell from comment #35)
> > What URL are you testing at?
>
> Ah. I was using:
>
> http://www-stage.mozilla.org/contribute/
>
> What's the login information for this staging site?
>
> http://mozilla-org-stage.silverorange.com/contribute/
>
Same as for the other silverorange dev servers, though it looks like now the http://www-stage.mozilla.org/contribute/ version is up-to-date. I don't get a blank page when submitting the form.
Whiteboard: r94653,94654,94737 b=staging → r94653,94654,94737,94752 b=staging
Comment 37•13 years ago
|
||
In r94752, changed the form submit anchor to the top of the #community-sub div, removed HTML5 required attributes due to issues in FF and Chrome. They can be added back later when said issues are addressed.
Reporter | ||
Comment 38•13 years ago
|
||
I'm seeing the inline errors now at
http://www-stage.mozilla.org/contribute/
Could we tweak the error text some to make things a little clear?
* Error for not selecting an area of interest: 'Please select an area of interest.'
* Error for not adding anything in the I'm interested in box: 'Please tell us more about your interests.'
I didn't see any response to my earlier question about the newsletter sign-up opt-in. Do you need anything from me for that?
Assignee | ||
Comment 39•13 years ago
|
||
(In reply to David Boswell from comment #38)
> I didn't see any response to my earlier question about the newsletter
> sign-up opt-in. Do you need anything from me for that?
David, James is almost ready with his new web-service for handling newsletter subscriptions across the various mozilla websites. I thought it best to wait for that and move ahead with everything else on this bug in the meantime rather than block on it. We can address that separately in Bug 649379 (immediately) after these changes, if that's ok with you.
Reporter | ||
Comment 40•13 years ago
|
||
Steven, thanks for the update on the newsletter subscription status. It's certainly fine with me to sequence the different parts in the way that makes the most sense.
Assignee | ||
Comment 41•13 years ago
|
||
(In reply to David Boswell from comment #38)
> Could we tweak the error text some to make things a little clear?
These messages have been updated with your text in the staging branch.
Whiteboard: r94653,94654,94737,94752 b=staging → r94653,94654,94737,94752,94802 b=staging
Reporter | ||
Comment 42•13 years ago
|
||
> These messages have been updated with your text in the staging branch.
Great. Things look good to go here with bug 568627, bug 569624 and bug 649382.
To get ready for deployment, I took out all of the placeholder response text except for Add-ons and Other. I'll work with the various point people once this is live to get the final text in. Let me know if I broke anything in r94844.
If my changes didn't break anything, please feel free to roll this out whenever you have a chance.
Comment 43•13 years ago
|
||
> Let me know if I broke anything in r94844.
>
Change looks good to me.
Assignee | ||
Updated•13 years ago
|
Whiteboard: r94653,94654,94737,94752,94802 b=staging → r94653,94654,94737,94752,94802,94844 b=staging
Updated•13 years ago
|
Target Milestone: 3.9 → 3.10
Assignee | ||
Comment 44•13 years ago
|
||
Merged these changes into trunk in r94886. This should be in production shortly.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 45•13 years ago
|
||
I'm seeing this on production now. The error messages are coming up fine. The customized emails were sent, although they ended up in my spam folder. Probably worth looking into this before rolling out auto-responses to all areas in the form.
Reporter | ||
Comment 46•13 years ago
|
||
FYI, I opened bug 685675 to be a new tracking bug for enhancements to the Want to help? form now that this bug is closed.
No longer depends on: 649379
Summary: Make a more permanent fix for the Get Involved contact form → Improve the usability of the Get Involved contact form
Comment 47•13 years ago
|
||
This needs QA before I can push it out.
Keywords: qawanted
Target Milestone: 3.10 → 3.11
Comment 48•13 years ago
|
||
Also, Steven, are the merges you made to stage after we recreated it?
Assignee | ||
Comment 49•13 years ago
|
||
(In reply to James Long (:jlongster) from comment #48)
> Also, Steven, are the merges you made to stage after we recreated it?
James, this is all in the mozilla.org repository, not mozilla.com - I think it's all set.
Comment 50•13 years ago
|
||
(In reply to Steven Garrity from comment #49)
> (In reply to James Long (:jlongster) from comment #48)
> > Also, Steven, are the merges you made to stage after we recreated it?
>
> James, this is all in the mozilla.org repository, not mozilla.com - I think
> it's all set.
Steven, are u using www-dev.allizom.org for staging changes to the .org repo
Keywords: qawanted
Assignee | ||
Comment 51•13 years ago
|
||
(In reply to raymond [:retornam] from comment #50)
> Steven, are u using www-dev.allizom.org for staging changes to the .org repo
I have not been - how does that work?
Comment 52•13 years ago
|
||
(In reply to Steven Garrity from comment #51)
> (In reply to raymond [:retornam] from comment #50)
> > Steven, are u using www-dev.allizom.org for staging changes to the .org repo
>
> I have not been - how does that work?
We have new dev servers, which were only recently setup. These reflect the new merged prod environment. I'll send an email about it.
http://www-dev.allizom.org/ -> trunk
http://www.allizom.org/ -> staging
These have both codebases running like normal, so try to use these from now on.
Comment 53•13 years ago
|
||
Of course, if you push to mozilla.org's trunk it still goes live immediately. :)
Updated•12 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
Updated•12 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•