Bug 872094 - avoid using cookies to store visible state of recipe results on recipe/job page
Summary: avoid using cookies to store visible state of recipe results on recipe/job page
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: web UI
Version: 0.9
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 23.0
Assignee: Dan Callaghan
QA Contact:
URL:
Whiteboard: ImplementationQuality
Depends On: 1263918
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-11-01 08:17 UTC by Amit Saha
Modified: 2018-02-06 00:41 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-07-07 23:10:41 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 674025 0 low CLOSED [RFE] Always show task specified by anchor 2021-02-22 00:41:40 UTC

Internal Links: 674025

Description Amit Saha 2012-11-01 08:17:51 UTC
Description of problem:

The current code for displaying the job details (/jobs/jobid) uses cookies (one per recipe) to maintain information regarding what results to display -- all,failed or none. 

While working on the solution to https://bugzilla.redhat.com/show_bug.cgi?id=674025, it was felt that a cookie-less solution may result in a better implementation to not only this bug, but the overall logic used in fetching the results for recipes. For example, if only a particular task's result is requested, it will only fetch the results of that recipe and not all the other recipes in the job.

The alternative suggested by Dan was using the History API.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:

See above.

Additional info:

Comment 4 Dan Callaghan 2016-03-23 02:59:36 UTC
This is implemented for the new recipe page design.

Expanded/collapsed tasks are reflected in the URL anchor, as in /recipes/123#task4,task5 with the usual fallback to localStorage in case the user re-opens the page with no anchor.

http://gerrit.beaker-project.org/4755

Comment 5 Mike McCune 2016-03-28 22:26:18 UTC
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

Comment 6 Dan Callaghan 2016-04-05 07:55:25 UTC
The results-reporting-improvements-take3 branch has been merged to develop.

Comment 9 Dan Callaghan 2016-07-07 23:10:41 UTC
Beaker 23.0 has been released.


Note You need to log in before you can comment on or make changes to this bug.