Feature #20850: Notification that workbench 1 is deprecated - Arvados

[X] Addressed all the points in the story specified, or can explains what changed
[X] Code is tested and passing, both automated and manual
[X] Documentation has been updated.
[n/a] Considered scaling issues, eg when happens when there are 10x - 10000000x more items than tested
[n/a] Considered backwards and forwards compatibility issues between client and server
[n/a] Considered GUI style guidelines

Add a box to the top of every workbench 1 page that tells the user to go to workbench 2. Sends the user to the same URL path as workbench 1 and relies on the wb1->wb2 redirects that were implemented in #20688

Actions

Copy link

#11

Updated by Brett Smith 9 months ago

Peter Amstutz wrote in #note-10:

20850-wb1-deprecated @ ee9d2efeaefb43fa77bdc567ddff0a46d3453a74

All low priority, in order of importance:

"workbench" should be capitalized both times it appears in the deprecation notice box text.

In both the blocks added to apps/workbench/app/views/layouts/body.html.erb, the indentation is inconsistent. There's both mixed tabs/spaces, and blocks out of alignment even when it's spaces-only.

I think the wb2_url block could be simplified down to wb2_url = URI::parse(Rails.configuration.Services.Workbench2.ExternalURL.to_s).merge(request.fullpath).to_s

Do what you want and then go ahead and merge, thanks.

Actions

Copy link

#12