action #36307
closedRemove limit option in Next & previous results tab UI
Description
Remove limit option in UI and set default to 100 next + 400 prev
See commnets in https://github.com/os-autoinst/openQA/pull/1614
Updated by mitiao over 6 years ago
- Related to action #13920: Showing context builds test result in the view of current "Previous Results" added
Updated by mkittler over 6 years ago
Ok, a few things which still needs enhancement in my opinion:
- When using the route
/tests/latestyou get a DataTable error. To reproduce, just start your web UI locally and open http://localhost:9526/tests/latest#next_previous. - It would be nice if the query parameter for the limit is updated as before. For that purpose I've once written the helper functions
parseQueryParams()andupdateQueryParams(params)inopenqa.js. - Reintroduce the link to the still valid URL for the latest job in general.
Updated by mitiao over 6 years ago
- Status changed from New to In Progress
Submitted PR https://github.com/os-autoinst/openQA/pull/1656
to fix DataTable error under /tests/latest and reintroduce the latest scenario URL
limit option will be in next PR
Updated by okurz over 6 years ago
mkittler wrote:
- Maybe reintroduce the URL for the latest job in general (if @okurz needs it).
That's weird wording. Think 1k bug reports on bugzilla.opensuse.org reference that link that is commonly used to find out if the latest one still fails. If this URL is now suddenly invalid it could cause quite some confusion and frustration.
Updated by mkittler over 6 years ago
PR is merged. I can confirm that the link is back and the error gone. So only the 2nd point missing.
@okurz Yes, the wording was wrong. The URL was never invalid, only the link has been removed.
Updated by mitiao over 6 years ago
PR for limit option:
https://github.com/os-autoinst/openQA/pull/1658
Updated by szarate over 6 years ago
- Target version changed from Current Sprint to Done