Bug #3170

[Workbench] Automatic reloading of pipeline instance page interferes with ajax loading of tabs - should just reload the components tab, not the whole page.

Added by Peter Amstutz almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench
Target version:
Start date:
07/18/2014
Due date:
% Done:

100%

Estimated time:
(Total: 1.00 h)
Story points:
1.0

Subtasks

Task #3292: Clean up tab loading javascriptResolvedTom Clegg

Task #3293: Modify periodic refresh to load only tab content, not page.ResolvedTom Clegg

Task #3291: Review 3170-refresh-tab-paneResolvedRadhika Chippada

Associated revisions

Revision d5b8f4ba
Added by Tom Clegg almost 6 years ago

Merge branch '3170-refresh-tab-pane'
closes #3170

Revision e8e0eecb (diff)
Added by Tom Clegg almost 6 years ago

3170: Fix stale use of tab_pane_valid_state. refs #3170

Revision 5fdfe4ca (diff)
Added by Tom Clegg almost 6 years ago

3170: Use find().all() instead of page.all() to accommodate ajax content. refs #3170, #2044

History

#1 Updated by Tom Clegg almost 6 years ago

  • Subject changed from Automatic reloading of pipeline instance page interferes with ajax loading of tabs. to Automatic reloading of pipeline instance page interferes with ajax loading of tabs - should just reload the components tab, not the whole page.

#2 Updated by Tom Clegg almost 6 years ago

  • Story points set to 1.0

#3 Updated by Tom Clegg almost 6 years ago

  • Target version set to 2014-08-06 Sprint

#4 Updated by Radhika Chippada almost 6 years ago

  • Subject changed from Automatic reloading of pipeline instance page interferes with ajax loading of tabs - should just reload the components tab, not the whole page. to [Workbench] Automatic reloading of pipeline instance page interferes with ajax loading of tabs - should just reload the components tab, not the whole page.
  • Category set to Workbench

#5 Updated by Tom Clegg almost 6 years ago

  • Assigned To set to Tom Clegg

#6 Updated by Tom Clegg almost 6 years ago

  • Status changed from New to In Progress

#7 Updated by Tom Clegg almost 6 years ago

3170-refresh-tab-pane @ 6a2406f

Includes some cleanup: move JavaScript out of views into tab_panes.js and pipeline_instances.js.

#8 Updated by Radhika Chippada almost 6 years ago

Everything looks great. The one comment is that it would be very helpful if you could add some comments to the newly created tab_panes.js file explaining its intended use.

#9 Updated by Anonymous almost 6 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:d5b8f4baf23f0b79e2f9b559b1da6f8898292f9f.

Also available in: Atom PDF