https://dev.arvados.org/https://dev.arvados.org/favicon.ico?15576888422023-04-18T14:20:09ZArvadosArvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1142712023-04-18T14:20:09ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/114271/diff?detail_id=111092">diff</a>)</li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1143072023-04-19T16:03:53ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Target version</strong> changed from <i>Future</i> to <i>Development 2023-04-26 sprint</i></li><li><strong>Assigned To</strong> set to <i>Stephen Smith</i></li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1144102023-04-25T13:51:15ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Target version</strong> changed from <i>Development 2023-04-26 sprint</i> to <i>Development 2023-05-10 sprint</i></li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1144412023-04-25T18:10:05ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Story points</strong> set to <i>2.0</i></li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1145642023-04-28T15:09:20ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/114564/diff?detail_id=111364">diff</a>)</li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1148212023-05-09T16:05:33ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/114821/diff?detail_id=111580">diff</a>)</li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1148222023-05-09T16:06:30ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/114822/diff?detail_id=111581">diff</a>)</li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1148692023-05-10T13:42:33ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Target version</strong> changed from <i>Development 2023-05-10 sprint</i> to <i>Development 2023-05-24 sprint</i></li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1149202023-05-10T15:40:24ZStephen Smith
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1149892023-05-10T21:01:44ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Release</strong> set to <i>63</i></li></ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1150032023-05-11T18:01:25ZStephen Smith
<ul></ul><p>Changes at <a class="changeset" title="20377: Removed unused imports Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>" href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/62f757227d45f1bbd338a0e27b22b8deb69f68b2">arvados-workbench2|62f757227d45f1bbd338a0e27b22b8deb69f68b2</a> branch 20377-dataexplorer-paging-bug<br />Tests <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/1215/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/1215/">developer-tests-workbench2: #1215 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=1215" alt="" /></a></a></p>
<ul>
<li>Rejects dataexplorer updates when update data page doesn't match current dataexplorer page
<ul>
<li>This allows the correct page to load through the dataexplorer middleware retry logic since the page number doesn't get clobbered</li>
</ul>
</li>
<li>Treats falsy values as 0 when performing the page comparison</li>
</ul> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1150102023-05-11T18:32:29ZPeter Amstutzpeter.amstutz@curii.com
<ul></ul><p>Stephen Smith wrote in <a href="#note-12">#note-12</a>:</p>
<blockquote>
<p>Changes at <a class="changeset" title="20377: Removed unused imports Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>" href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/62f757227d45f1bbd338a0e27b22b8deb69f68b2">arvados-workbench2|62f757227d45f1bbd338a0e27b22b8deb69f68b2</a> branch 20377-dataexplorer-paging-bug<br />Tests <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/1215/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/1215/">developer-tests-workbench2: #1215 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=1215" alt="" /></a></a></p>
<ul>
<li>Rejects dataexplorer updates when update data page doesn't match current dataexplorer page
<ul>
<li>This allows the correct page to load through the dataexplorer middleware retry logic since the page number doesn't get clobbered</li>
</ul>
</li>
<li>Treats falsy values as 0 when performing the page comparison</li>
</ul>
</blockquote>
<p>This LGTM.</p>
<p>But to note for future improvement, it would be nice if it could buffer the clicks a little bit so if I rapidly click 10 times to go 10 pages ahead it doesn't send requests for all 10 pages.</p> Arvados - Bug #20377: Buggy process panel paging https://dev.arvados.org/issues/20377?journal_id=1150172023-05-12T15:29:02ZStephen Smith
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul><p>Merged in <a class="changeset" title="Merge branch '20377-dataexplorer-paging-bug' refs #20377 Arvados-DCO-1.1-Signed-off-by: Peter Am..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/7f0ee8806944ff39e3545572ebad8f63b8e0938d">arvados-workbench2|7f0ee8806944ff39e3545572ebad8f63b8e0938d</a></p>