Add MOTD to editor tools

VERIFIED FIXED in 5.1

Status

addons.mozilla.org Graveyard
Admin/Editor Tools
P2
normal
VERIFIED FIXED
9 years ago
2 years ago

People

(Reporter: osunick, Assigned: abuchanan)

Tracking

unspecified

Details

(URL)

Attachments

(4 attachments)

(Reporter)

Description

9 years ago
We need the ability to set a message that appears at the top of every review screen that can be set by an administrator.  This message would contain information like 

"Be on the lookout for add-ons which change search preferences, for more info read our wiki."

Only admins and senior editors can change this message from the admin tools, and we need the ability to put links in this message.

This should be placed above the title bar that starts with "Review <add-on name" in bold text.
Assignee: nobody → buchanae
Priority: -- → P2
(Assignee)

Comment 1

9 years ago
Created attachment 400817 [details] [diff] [review]
patch v1
Attachment #400817 - Flags: review?(rdoherty)
Comment on attachment 400817 [details] [diff] [review]
patch v1

Looks good!
Attachment #400817 - Flags: review?(rdoherty) → review+
Comment on attachment 400817 [details] [diff] [review]
patch v1

Small note: a blank line between motd(), _set_motd() and _get_motd() would be nice :)
(Assignee)

Comment 4

9 years ago
patch committed,

ending        app/controllers/editors_controller.php
Adding         app/views/editors/motd_admin.thtml
Sending        app/views/editors/review.thtml
Sending        app/views/elements/developers/editorsmenu.thtml
Adding         app/views/elements/developers/review_motd.thtml
Sending        app/webroot/css/editors.css

The rule "Admin:EditorsMOTD" needs to be added to the Senior Editor group when deployed.  I've added this on preview.

The MOTD can be edited at,
https://preview.addons.mozilla.org/en-US/editors/motd

(In reply to comment #3)
> (From update of attachment 400817 [details] [diff] [review])
> Small note: a blank line between motd(), _set_motd() and _get_motd() would be
> nice :)

done
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Verified FIXED on https://preview.addons.mozilla.org/en-US/editors/motd -- I didn't test for RTL.  Ehsan, is this OK in RTL?

Comment 6

9 years ago
(In reply to comment #5)
> Verified FIXED on https://preview.addons.mozilla.org/en-US/editors/motd -- I
> didn't test for RTL.  Ehsan, is this OK in RTL?

Can you please attach screenshots?  I don't have access to the editors panel in my account on preview.a.m.o.
(Assignee)

Comment 7

9 years ago
Created attachment 401048 [details]
RTL screenshot 1
(Assignee)

Comment 8

9 years ago
Created attachment 401050 [details]
RTL screenshot 2
Attachment #401048 - Attachment mime type: application/octet-stream → image/png
Attachment #401050 - Attachment mime type: application/octet-stream → image/png

Comment 9

9 years ago
Comment on attachment 401050 [details]
RTL screenshot 2

The first screenshot looks correct.

This one, however, seems to have problems.  It would be great if you can attach an en-US screenshot of the same page so that I can compare them exactly.
(Assignee)

Comment 10

9 years ago
Created attachment 401307 [details]
screenshot, english
(Assignee)

Comment 11

9 years ago
keep in mind, the only change to the review page (i.e. screenshots 2 & 3) is the blue box at the top.  Any other RTL issues are most likely a existing issue -> separate bug.

Also, if it would be helpful, I think we could grant you the Senior Editor role on preview, for testing.  Clouserw, is that ok?  (note, the DB refreshes every day, so you'll lose this role when it does)

Thanks!
The preview database hasn't refreshed in months.  I added Ehsan as a senior editor.

Comment 13

9 years ago
(In reply to comment #11)
> keep in mind, the only change to the review page (i.e. screenshots 2 & 3) is
> the blue box at the top.  Any other RTL issues are most likely a existing issue
> -> separate bug.

Oh, the blue bar is fine.  I'll file another bug for the rest of the issues.
Status: RESOLVED → VERIFIED

Comment 14

9 years ago
(In reply to comment #13)
> (In reply to comment #11)
> > keep in mind, the only change to the review page (i.e. screenshots 2 & 3) is
> > the blue box at the top.  Any other RTL issues are most likely a existing issue
> > -> separate bug.
> 
> Oh, the blue bar is fine.  I'll file another bug for the rest of the issues.

Filed bug 518156.
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.