Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(346)

Issue 2523433002: Add link to old ui back in (Closed)

Created:
4 years, 1 month ago by kjlubick
Modified:
4 years, 1 month ago
Reviewers:
M-A Ruel
CC:
chromium-reviews, infra-reviews+luci-py_chromium.org
Target Ref:
refs/heads/master
Project:
luci-py
Visibility:
Public.

Description

Add link to old ui back in This keeps the new ui as default (/user/* and /restricted/* will still redirect to new ui), but also adds in a way to easily see the old ui. The old UI will go away next year. BUG=664829 Committed: https://github.com/luci/luci-py/commit/8a4efd26384614855c0ffb511b51c21d0aad4e1b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2126 lines, -27 lines) Patch
M appengine/swarming/handlers_frontend.py View 3 chunks +623 lines, -23 lines 0 comments Download
A appengine/swarming/templates/bot_view.html View 1 chunk +60 lines, -0 lines 0 comments Download
A appengine/swarming/templates/restricted_bot.html View 1 chunk +228 lines, -0 lines 0 comments Download
A appengine/swarming/templates/restricted_botslist.html View 1 chunk +187 lines, -0 lines 0 comments Download
M appengine/swarming/templates/root.html View 1 chunk +2 lines, -2 lines 0 comments Download
A appengine/swarming/templates/user_task.html View 1 chunk +622 lines, -0 lines 0 comments Download
A appengine/swarming/templates/user_tasks.html View 1 chunk +375 lines, -0 lines 0 comments Download
M appengine/swarming/ui/build/elements.html View 1 chunk +1 line, -1 line 0 comments Download
M appengine/swarming/ui/res/imp/common/swarming-app.html View 2 chunks +27 lines, -0 lines 0 comments Download
M appengine/swarming/ui/res/imp/index/swarming-index-demo.html View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (5 generated)
kjlubick
See it on https://2495-74a951a-tainted-kjlubick-dot-chromium-swarm-dev.appspot.com e.g. https://2495-74a951a-tainted-kjlubick-dot-chromium-swarm-dev.appspot.com/oldui/restricted/bot/skia-rpi-test
4 years, 1 month ago (2016-11-21 17:06:02 UTC) #3
M-A Ruel
rs lgtm
4 years, 1 month ago (2016-11-21 17:43:12 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2523433002/1
4 years, 1 month ago (2016-11-21 17:45:27 UTC) #6
commit-bot: I haz the power
4 years, 1 month ago (2016-11-21 17:49:28 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://github.com/luci/luci-py/commit/8a4efd26384614855c0ffb511b51c21d0aad4e1b

Powered by Google App Engine
This is Rietveld 408576698