Bug #19337

[UX] repo discovery footer for page listing malformed

Added by Thomas McKay about 1 year ago. Updated 7 days ago.

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

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 1 year 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 1 year ago

  • Legacy Backlogs Release (now unused) set to 114

#3 Updated by Thomas McKay about 1 year ago

  • Target version changed from 185 to 195

#4 Updated by Eric Helms about 1 year ago

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

#5 Updated by Justin Sherrill about 1 year ago

  • Assignee changed from Thomas McKay to Walden Raines

#6 Updated by Justin Sherrill 12 months ago

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

#7 Updated by Eric Helms 12 months ago

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

#8 Updated by Eric Helms 12 months ago

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

#9 Updated by Walden Raines 10 months ago

  • Target version changed from 195 to 219

#10 Updated by Walden Raines 10 months ago

  • Assignee deleted (Walden Raines)

#11 Updated by Walden Raines 29 days ago

  • Target version deleted (219)

Also available in: Atom PDF