Project

General

Profile

Bug #19337

[UX] repo discovery footer for page listing malformed

Added by Thomas McKay about 2 years ago. Updated 11 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Repositories
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

The page information at the bottom of discovered repos is missing values (shows NaN, missing page numbers, etc.)

To reproduce go to repo discovery, docker type, "registry.access.redhat.com", then note page info at bottom of ui page.

History

#1 Updated by Walden Raines about 2 years ago

  • Assignee changed from Walden Raines to Thomas McKay

I wonder if this is related to recent docker tags changes. The NaN, missing page numbers, etc. is caused by total and or subtotal not being set.

It looks like these should be set in an atypical way here:

https://github.com/katello/katello/blob/master/engines/bastion_katello/app/assets/javascripts/bastion_katello/products/discovery/discovery.controller.js#L49-L53

Reassigning to Tom since he touched this last and has more familiarity with this code.

#2 Updated by John Mitsch about 2 years ago

  • Legacy Backlogs Release (now unused) set to 114

#3 Updated by Thomas McKay about 2 years ago

  • Target version changed from 185 to 195

#4 Updated by Eric Helms almost 2 years ago

  • Legacy Backlogs Release (now unused) changed from 114 to 267

#5 Updated by Justin Sherrill almost 2 years ago

  • Assignee changed from Thomas McKay to Walden Raines

#6 Updated by Justin Sherrill almost 2 years ago

  • Legacy Backlogs Release (now unused) changed from 267 to 281

#7 Updated by Eric Helms almost 2 years ago

  • Legacy Backlogs Release (now unused) changed from 281 to 286

#8 Updated by Eric Helms almost 2 years ago

  • Legacy Backlogs Release (now unused) changed from 286 to 114

#9 Updated by Walden Raines over 1 year ago

  • Target version changed from 195 to 219

#10 Updated by Walden Raines over 1 year ago

  • Assignee deleted (Walden Raines)

#11 Updated by Walden Raines 12 months ago

  • Target version deleted (219)

Also available in: Atom PDF