refactor auslib/web/views/releases.py

NEW
Unassigned

Status

Release Engineering
Balrog: Backend
P5
normal
4 years ago
20 days ago

People

(Reporter: bhearsum, Unassigned)

Tracking

(Depends on: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [lang=python])

(Reporter)

Description

4 years ago
In hindsight, I've realized that https://github.com/mozilla/balrog/blob/master/auslib/admin/views/releases.py is terrible to read. It's full of confusing names, unnecessary callbacks, and other badness. I wrote it that way to reduce code duplication as much as possible but I think it's pretty clear now that it's not worth it. There might be a better way to factor out common things and be less ugly, but I think this file would be better off with duplicate code + easier readability than its current state.
(Assignee)

Updated

4 years ago
Product: mozilla.org → Release Engineering
(Reporter)

Comment 1

4 years ago
mass component change
Component: General Automation → Balrog: Backend
(Reporter)

Comment 2

3 years ago
bug 1021021 is going to add a schema which is completely different than anything we currently have. It might be a good time to do some or all of this.

Updated

10 months ago
Depends on: 1313742
(Reporter)

Comment 3

7 months ago
I'm going to leave this open for now, but it's likely to go away when we switch to Swagger.
(Reporter)

Updated

7 months ago
Whiteboard: [balrog] → [lang=python]
(Reporter)

Updated

20 days ago
Priority: P3 → P5
You need to log in before you can comment on or make changes to this bug.