790798 - Add L10n Dashboard notifications/messages

Status: RESOLVED FIXED

(support.mozilla.org :: Knowledge Base Software, task, P2)

Description

[Kadir Topal [:atopal]]

* As a localizer I would like to receive all importat messages in the l10n  dashboard. I would like to be made aware of important communications  there.

* As a KB manager, community manager or locale leader, I would like to be  able to send imporntant information to all the localizers in one local  through the dahsboard

    https://sumo.etherpad.mozilla.org/103

Comment 1

[Bram Pitoyo [:bram]]

For the design of this notification, we should use the style found on <div class="warning">

This is the same style found on the yellow warning box on, for example, the Private Browsing KB article page:
https://support.mozilla.org/en-US/kb/private-browsing-browse-web-without-saving-info

Comment 2

[Ricky Rosario [:rrosario, :r1cky]]

I am not sure how we are going to map groups to a locale. I need to look at this.

Comment 3

[Ricky Rosario [:rrosario, :r1cky]]

I think we should be able to add a nullable FK from the Announcement model to the Locale model. Then add the UI to the locale dashboards.

Comment 4

[Ricky Rosario [:rrosario, :r1cky]]

I made this a 3pter, it might be a 2 but I am being conservative in case my idea doesn't work.

Comment 5

[Michael Cooper [:mythmon]]

Looking into this, we already have the ability to send announcements to a group, and there appear to be groups for each localization language. In fact, on http://support.mozilla.org/dashboards/10 (the German localization dashboard) there is an announcements that Kadir added.

Is this functionality sufficient? There is still no UI to add an announcement, which limits the users that can create an announcement to those that have access to the admin, but this is fixable. It seems that we simply need to a form for users without access to the admin (but with the right privelages) to be able to add announcements, and to minorly restyle the announcements.

Is this correct?

Comment 6

[Ricky Rosario [:rrosario, :r1cky]]

To clarify, this bug refers to the L10n dashboard that lives at /<locale>/localization and not to the other group dashboards that can be related to locale groups.

We are trying to move away from having to create a group for every locale+role (leaders, reviewers, editors,...), so we created the Locale model. We should use it to assign announcements to locales.

Also, we currently require the admin to create announcements. Locale leaders don't have admin access so we would have to create new forms+views+ui+tests to do this. If it is becomes too much, please spin off a new bug.

Comment 7

[Michael Cooper [:mythmon]]

Backend for this: https://github.com/mozilla/kitsune/pull/1018
Landed in: 8af3579c242887a837c91eeae7f60a3822a79cc0

I'm working on the front end today.

Comment 8

[Michael Cooper [:mythmon]]

PR: https://github.com/mozilla/kitsune/pull/1023
Landed in: https://github.com/mozilla/kitsune/commit/f67ea61de6b028db41b0432dd221e1023d551e7c

Production is frozen for the new year, so will push this next week.

Comment 9

[Ricky Rosario [:rrosario, :r1cky]]

Deployed to prod.