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

RESOLVED FIXED

Status

()

bugzilla.mozilla.org
User Interface
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dkl, Assigned: dkl)

Tracking

Production

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
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
(Assignee)

Comment 1

6 years ago
Created attachment 569075 [details] [diff] [review]
survey announcement content (v1)

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

dkl
Attachment #569075 - Flags: review?(glob)
(Assignee)

Comment 2

6 years ago
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.
Attachment #569075 - Attachment is obsolete: true
Attachment #569075 - Flags: review?(glob)
(Assignee)

Comment 3

6 years ago
Created attachment 569094 [details] [diff] [review]
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
Attachment #569094 - Flags: review?(glob)
Do you have a release process with someone from IT or do you need me to file a push bug?
(Assignee)

Comment 5

6 years ago
(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 on attachment 569094 [details] [diff] [review]
Patch to add survey announcement to top of BMO pages (v2)

r=glob
Attachment #569094 - Flags: review?(glob) → review+
(Assignee)

Comment 7

6 years ago
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
OS: Linux → All
Hardware: x86_64 → All
(Assignee)

Comment 9

6 years ago
(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
WFM; lets do it.
(Assignee)

Comment 11

6 years ago
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://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.0
modified template/en/default/global/header.html.tmpl
Committed revision 7910

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

Comment 13

6 years ago
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
Depends on: 697137
Sounds good, we'll keep the survey link live until it goes down from Bugzilla.
(Assignee)

Comment 15

6 years ago
This has been pushed live now. Closing.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
removed:

Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.0/
modified template/en/default/global/header.html.tmpl
Committed revision 7920.
You need to log in before you can comment on or make changes to this bug.