864394 - create user interface for manipulating skip lists

Status: RESOLVED INVALID

(Socorro :: Webapp, task)

Description

[K Lars Lohn [:lars] [:klohn]]

the skip list rules are to be moved from configuration into the database.

create a Web interface that allows an authorized user to add/remove/edit skip list category  and rule entries.   See Bug 864391 for details on the database table structure and the Bug 864392 for the RESTful middleware API details.

Comment 1

[Robert Kaiser]

FYI, bug 528390 has existed for a long time for this, just with a summary that nobody can query for, I guess (no "skip" or "admin" in it, for example). ;-)

Comment 2

[[github robot]]

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

https://github.com/mozilla/socorro/commit/821411aa2ce39a8c850cef2fb1ed0db168817a21
fixes bug 864394 - create user interface for manipulating skip lists, r=AdrianGaudebert

Comment 3

[Matt Brandt [:mbrandt]]

Attachment: qa - reopened

Reopening for clarification (testing against stage):
1. I don't see any pre-existing skiplist rules being populated on stage. Should there be?

2. Delete does not appear to work - I created a new rule and then attempted to delete it. https://errormill.mozilla.org/webtools/socorro-stage/group/95481/

3. Table boarders appear to be missing - see screenshot

Comment 4

[Peter Bengtsson [:peterbe]]

All good points. I think I too eagerly closed this.

1. I'll need to talk to lars and selena about doing a first move of existing skiplist over. This is a SQL thing and unrelated to the UI but thanks for pointing it out.

2. Darn! It's because we use the old middleware on stage. Let me get back to you once I know when we upgrade the new middleware. 

3. That's the way our tables work/look. See https://crash-stats.mozilla.com/admin/featured-versions/
It just looks a bit worse on this page because of the extra <tfoot> row.

Comment 5

[Peter Bengtsson [:peterbe]]

See https://bugzilla.mozilla.org/show_bug.cgi?id=864381#c2