696770 - Add announcehtml text to ask people to fill out contributor survey on BMO

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: User Interface, defect)

Description

[David Lawrence [:dkl]]

Aakash was unable to email a list of Bugzilla accounts due to permission not being previously given to receive such announcements. So instead we want to add
a temporary announcehtml message to the top of each page asking logged in users to fill out a contributor survey. The message will have the following requirements.

1) Logged in users only. 
2) Not be too invasive
3) Hide the announcement once the link is clicked or the user declines it.
4) Set a cookie so the announcement no longer displays after hiding.

Normally announcehtml is used for BMO administrative stuff such as outages, upgrades, etc and we should not leave this in for too long. 1-2 weeks be sufficient?

Aakash, please supply the actual link to add to the text once you can get it.

Attachment coming.

dkl

Comment 1

[David Lawrence [:dkl]]

Attachment: survey announcement content (v1)

This is the text that would go in data/params announcehtml.

dkl

Comment 2

[David Lawrence [:dkl]]

Comment on attachment 569075 [details] [diff] [review]
survey announcement content (v1)

Ignore this one. Need to add it the code to check for being loggged in or not.

Comment 3

[David Lawrence [:dkl]]

Attachment: Patch to add survey announcement to top of BMO pages (v2)

New patch that actually puts the announcement in the template code instead of in the announcehtml. This was because javascript is unable to access login cookies so I cannot tell if the user is logged in or not that way. Using the templates, I can. Downside is this will require a code push by IT to add this to the site and then to remove it later. I think this should be ok although we normally only do code pushes for non-critical fixes on each Thursday.

dkl

Comment 4

[Aakash Desai [:aakashd]]

Do you have a release process with someone from IT or do you need me to file a push bug?

Comment 5

[David Lawrence [:dkl]]

(In reply to Aakash Desai [:aakashd] from comment #4)
> Do you have a release process with someone from IT or do you need me to file
> a push bug?

Yes once we are sure all is working properly, we just file a bug report for the code push.

dkl

Comment 6

[:glob ✱]

Comment on attachment 569094 [details] [diff] [review]
Patch to add survey announcement to top of BMO pages (v2)

r=glob

Comment 7

[David Lawrence [:dkl]]

Thanks for the review.

Aakash, I just need from you the full url to the survey itself to put in the code before
we push live. 

dkl

Comment 8

[Aakash Desai [:aakashd]]

The URL is http://mzl.la/pCnv4Q

Comment 9

[David Lawrence [:dkl]]

(In reply to Aakash Desai [:aakashd] from comment #8)
> The URL is http://mzl.la/pCnv4Q

Thanks. I have pushed the latest patch to our test instance at 

https://bugzilla-stage-tip.mozilla.org/

Please go there and give it a try. You should see it first time and if you hide it and want to see it again just delete the 'surveyRead' cookie for the same domain in your browser prefs.

Let me know if it is ok and I will get it pushed out tonight/tomorrow hopefully.

dkl

Comment 10

[Aakash Desai [:aakashd]]

WFM; lets do it.

Comment 11

[David Lawrence [:dkl]]

Thanks. I will create the bug to get this pushed out hopefully today.

Aakash, we also need to set a hard date that this can be removed. How long would you like this to be visible and what Thursday update can I go ahead and schedule to have it removed?

Committing to: bzr+ssh://[e-mail]/bmo/4.0
modified template/en/default/global/header.html.tmpl
Committed revision 7910

dkl

Comment 12

[Aakash Desai [:aakashd]]

Keep it open just for the end of this week (Friday, 11:50pm Pacific).

Comment 13

[David Lawrence [:dkl]]

Ok. I will put in another bug to have the code push done again without the survey announcement. It may go over to Monday though since the time you request is after hours for the IT dept.

dkl

Comment 14

[Aakash Desai [:aakashd]]

Sounds good, we'll keep the survey link live until it goes down from Bugzilla.

Comment 15

[David Lawrence [:dkl]]

This has been pushed live now. Closing.

Comment 16

[:glob ✱]

removed:

Committing to: bzr+ssh://[e-mail]/bmo/4.0/
modified template/en/default/global/header.html.tmpl
Committed revision 7920.