If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

consolidate paginator uses

RESOLVED FIXED in 2014-10-21

Status

Marketplace
Code Quality
P4
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: eviljeff, Assigned: robhudson)

Tracking

2014-Q4
2014-10-21
x86_64
Windows 7
Points:
---
Dependency tree / graph

Details

(Whiteboard: [repoman][qa-])

(Reporter)

Description

3 years ago
In various places the django paginator, a pre-impala paginator, and a post-impala paginator are used (with the accompanying templates in the latter cases).  Surely no project needs that many...

Updated

3 years ago
Priority: -- → P4
(Reporter)

Updated

3 years ago
Blocks: 1074184
(Assignee)

Updated

3 years ago
Whiteboard: [repoman]
(Assignee)

Comment 1

3 years ago
Removed one of the paginator classes:
https://github.com/mozilla/zamboni/commit/ae65b72
(Assignee)

Updated

3 years ago
Assignee: nobody → robhudson.mozbugs
(Assignee)

Comment 2

3 years ago
Removed pre-impala paginator. There should now only be a Paginator and ESPaginator class, and a impala_paginator template helper.

https://github.com/mozilla/zamboni/commit/273cbf6
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Whiteboard: [repoman] → [repoman][qa-]
Target Milestone: --- → 2014-10-21
(Assignee)

Comment 3

3 years ago
Actually this is the right commit:
https://github.com/mozilla/zamboni/commit/94ffd16b
You need to log in before you can comment on or make changes to this bug.