https://dev.arvados.org/https://dev.arvados.org/favicon.ico?15576888422015-10-19T20:26:12ZArvadosArvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=315402015-10-19T20:26:12ZBrett Smithbrett.smith@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/31540/diff?detail_id=30991">diff</a>)</li></ul> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=315822015-10-20T18:31:34ZBrett Smithbrett.smith@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/31582/diff?detail_id=31019">diff</a>)</li><li><strong>Story points</strong> set to <i>0.5</i></li></ul> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=316042015-10-21T13:27:52ZBrett Smithbrett.smith@curii.com
<ul><li><strong>Target version</strong> set to <i>Arvados Future Sprints</i></li></ul> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=331432015-12-03T18:34:13ZNancy Ouyangnancy@curoverse.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li><li><strong>Assigned To</strong> set to <i>Nancy Ouyang</i></li></ul><p>I am working on this with Radhika's help.</p> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=334372015-12-11T23:15:22ZNancy Ouyangnancy@curoverse.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>New</i></li></ul><p>I couldn't get the Workbench tests to run*, so I was not able to write tests for my code. I pushed the two or three lines of logic I added to branch 7607-getting-started-flag in case it's of use.</p>
<ul>
<li>Kept running into:<br />[keepproxy] 2015/12/11 13:28:17 Error setting up arvados client Missing required environment variable ARVADOS_API_HOST</li>
</ul>
<p>Sorry I couldn't wrap up this ticket, and thanks to Radhika for getting me as far into running the tests as I got!</p> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=338392016-01-04T19:40:27ZRadhika Chippadaradhika@curoverse.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li><li><strong>Assigned To</strong> changed from <i>Nancy Ouyang</i> to <i>Radhika Chippada</i></li></ul><p>Nancy's commit at <a class="changeset" title="7607: enablee flag should be set to false by default" href="https://dev.arvados.org/projects/arvados/repository/arvados/revisions/dfbdc1ba1ae2d41aec76f6bd7a7afe538419a3f9">dfbdc1ba1ae2d41aec76f6bd7a7afe538419a3f9</a> is good. The Getting started popup is shown on first access of workbench and the menu option is included in Help menu only when enabled. The integration tests are failing and need to be updated, and additional tests need to be added.</p> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=338612016-01-05T13:45:59ZRadhika Chippadaradhika@curoverse.com
<ul></ul><p>Branch 7607-getting-started-flag ready for review. This includes:</p>
<ul>
<li>Nancy's original code updates, where a config param is added to enable the getting started popup</li>
</ul>
<ul>
<li>Added / updated workbench tests to meet the new default behavior.</li>
</ul>
<ul>
<li>application_layout_test updates
<ul>
<li>The previous tests are updated to not expect the getting started popup (with the default config of disabled)</li>
<li>Added a new test with config enabled and expect the popup. Also added a new user with profile but no getting_started_shown property to test both flavors (user has seen it before + not yet seen)</li>
<li>I also noticed that I had three different sets of tests doing check_help_menu, verify_homepage, verify_system_menu. These checks did not need separate tests doing the whole page loading all over. I consolidated these into a single test set and brought down the test run time from 90s to 60s.</li>
</ul></li>
</ul> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=339062016-01-05T19:52:45ZPeter Amstutzpeter.amstutz@curii.com
<ul></ul><p>7607-getting-started-flag LGTM</p> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=339232016-01-05T22:28:55ZRadhika Chippadaradhika@curoverse.com
<ul><li><strong>Target version</strong> changed from <i>Arvados Future Sprints</i> to <i>2016-01-06 sprint</i></li></ul> Arvados - Bug #7607: [Workbench] "Getting Started" popup is only appropriate for the public betahttps://dev.arvados.org/issues/7607?journal_id=339312016-01-05T22:40:11ZRadhika Chippadaradhika@curoverse.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul><p>Applied in changeset arvados|commit:e1276d6ebfbf073b747825cfcbfdd0054b95efa4.</p>