997275 - Build event metrics management interface

Status: VERIFIED FIXED

(Mozilla Reps Graveyard :: reps.mozilla.org, task)

Description

[Justin Crawford [:hoosteeno] [:jcrawford]]

We're anticipating that bug 997271 will ask for positive integer metrics only. Assuming that let's build a way for the reps council to create/edit/delete event metrics. 

* As a reps council member (or admin), I should be able to navigate to Portal settings -> Event metrics to see a list of event metrics. I should be able to edit or delete existing ones and add new ones.

* As a reps council member (or admin), when I create or edit an event metric, I will be presented with a form that allows me to input the name of a positive-integer metric (example: "Bugs closed"). When I submit the form that metric will be added to the list of metrics a rep can choose when creating an event (note: adding these metrics to event creation will be a separate bug).

Comment 1

[Justin Crawford [:hoosteeno] [:jcrawford]]

See the list in https://bugzilla.mozilla.org/show_bug.cgi?id=997271#c3 for the initial list of event metrics.

Comment 2

[[github robot]]

Commit pushed to master at https://github.com/mozilla/remo

https://github.com/mozilla/remo/commit/26ce3fa7ca113e67c65a810a20b4b7b7c977b205
[Fix bug 997275] Build event metrics management interface.

* Remove old metrics related_name.
* Add EventMetric and EventMetricOutcome models.
* Show EventMetric to base content.
* Fix /stats/ URL conflict with admin URL.
* Register new models to admin.
* Minor fixes in new models Meta.
* Replace old metrics with new metrics.
* Remove unused code from old metrics.
* Change templates to handle new metrics.
* Minor compatibility fixes between old and new metrics.
* Add factory for new metrics

Comment 3

[Tasos Katsoulas [:tasos]]

Reopening this bug because only admins are able to add/edit new event metrics.

Comment 4

[[github robot]]

Commits pushed to master at https://github.com/mozilla/remo

https://github.com/mozilla/remo/commit/381b7b9b55392809c485d488fe9b0db390056453
[Fix bug 997275] Add dynamic groups to PermissionMixin.

https://github.com/mozilla/remo/commit/02af0eab33ad10e881af2d978190a824bb90eabe
Merge pull request #755 from johngian/fix-metrics-permissions

[Fix bug 997275] Add dynamic groups to PermissionMixin.

Comment 5

[Tasos Katsoulas [:tasos]]

Verified on stage.

Steps:
* Logged in as a Council member (non-admin account).
* Added a new event metric through the Portal settings page.
* Non-council account produces a permission denied message.