Move elements/ to ui/

Move elements/ to ui/

Update the app.yaml to point to the correct files.

Also updated any Polymer/Bower/NPM dependencies for the build.

BUG=631047
Committed: https://github.com/luci/luci-py/commit/d48d94687f6961e6b69379987af5a480153b5071

[kjlubick, 2016-10-10 16:43:14]

Description was changed from

==========
Move elements/ to ui/

Also update the app.yaml and related places

BUG=
==========

to

==========
Move elements/ to ui/

Also update the app.yaml and related places

BUG=631047
==========

[kjlubick, 2016-10-10 16:43:14]

kjlubick@google.com changed reviewers:
+ jcgregorio@google.com, stephana@google.com

[kjlubick, 2016-10-10 17:06:15]

Description was changed from

==========
Move elements/ to ui/

Also update the app.yaml and related places

BUG=631047
==========

to

==========
Move elements/ to ui/

Update the app.yaml to point to the correct files.

Also updated any Polymer/Bower/NPM dependencies for the build.

BUG=631047
==========

[jcgregorio, 2016-10-10 17:32:36]

https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml
File appengine/swarming/app.yaml (right):

https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml...
appengine/swarming/app.yaml:46: - url: /ui/polymer05/([^/]+\-build)\.(html|js)
Is polymer05 represent Polymer 0.5? If so, should this be updated to polymer1?

[kjlubick, 2016-10-10 17:47:25]

On 2016/10/10 at 17:32:36, jcgregorio wrote:
> https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml
> File appengine/swarming/app.yaml (right):
> 
>
https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml...
> appengine/swarming/app.yaml:46: - url: /ui/polymer05/([^/]+\-build)\.(html|js)
> Is polymer05 represent Polymer 0.5? If so, should this be updated to polymer1?

That is correct.  There is a tiny amount of stats stuff that is still in Polymer
0.5  However, that's a much fuzzier yak to shave than I would like.  M-A says
it's fine to push the stats stuff off to v2, because of the additional steps it
would take (i.e. server side changes, some auth fixes, etc).

I'll still be keeping the old UI around immediately after the transition, just
in case.

[jcgregorio, 2016-10-10 17:56:21]

On 2016/10/10 at 17:47:25, kjlubick wrote:
> On 2016/10/10 at 17:32:36, jcgregorio wrote:
> >
https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml
> > File appengine/swarming/app.yaml (right):
> > 
> >
https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml...
> > appengine/swarming/app.yaml:46: - url:
/ui/polymer05/([^/]+\-build)\.(html|js)
> > Is polymer05 represent Polymer 0.5? If so, should this be updated to
polymer1?
> 
> That is correct.  There is a tiny amount of stats stuff that is still in
Polymer 0.5  However, that's a much fuzzier yak to shave than I would like.  M-A
says it's fine to push the stats stuff off to v2, because of the additional
steps it would take (i.e. server side changes, some auth fixes, etc).
> 
> I'll still be keeping the old UI around immediately after the transition, just
in case.

Seems error prone, but if everyone is OK with it.

[jcgregorio, 2016-10-10 17:56:21]

On 2016/10/10 at 17:47:25, kjlubick wrote:
> On 2016/10/10 at 17:32:36, jcgregorio wrote:
> >
https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml
> > File appengine/swarming/app.yaml (right):
> > 
> >
https://codereview.chromium.org/2408743002/diff/1/appengine/swarming/app.yaml...
> > appengine/swarming/app.yaml:46: - url:
/ui/polymer05/([^/]+\-build)\.(html|js)
> > Is polymer05 represent Polymer 0.5? If so, should this be updated to
polymer1?
> 
> That is correct.  There is a tiny amount of stats stuff that is still in
Polymer 0.5  However, that's a much fuzzier yak to shave than I would like.  M-A
says it's fine to push the stats stuff off to v2, because of the additional
steps it would take (i.e. server side changes, some auth fixes, etc).
> 
> I'll still be keeping the old UI around immediately after the transition, just
in case.

Seems error prone, but if everyone is OK with it.

[jcgregorio, 2016-10-10 17:56:27]

lgtm

[stephana, 2016-10-10 18:56:55]

On 2016/10/10 17:56:27, jcgregorio wrote:
> lgtm

lgtm

[kjlubick, 2016-10-13 13:02:02]

The CQ bit was checked by kjlubick@google.com

[kjlubick, 2016-10-13 13:02:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
jcgregorio@google.com, stephana@google.com
Link to the patchset: https://codereview.chromium.org/2408743002/#ps20001
(title: "rebase again")

[commit-bot: I haz the power, 2016-10-13 13:02:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-13 13:02:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-13 13:02:15]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[KevinL, 2016-10-13 13:05:55]

kjlubick@chromium.org changed reviewers:
+ kjlubick@chromium.org

[KevinL, 2016-10-13 13:05:56]

lgtm

[KevinL, 2016-10-13 13:05:58]

The CQ bit was checked by kjlubick@chromium.org

[commit-bot: I haz the power, 2016-10-13 13:06:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-13 13:09:56]

Description was changed from

==========
Move elements/ to ui/

Update the app.yaml to point to the correct files.

Also updated any Polymer/Bower/NPM dependencies for the build.

BUG=631047
==========

to

==========
Move elements/ to ui/

Update the app.yaml to point to the correct files.

Also updated any Polymer/Bower/NPM dependencies for the build.

BUG=631047

Committed:
https://github.com/luci/luci-py/commit/d48d94687f6961e6b69379987af5a480153b5071
==========

[commit-bot: I haz the power, 2016-10-13 13:09:58]

Committed patchset #3 (id:20001) as
https://github.com/luci/luci-py/commit/d48d94687f6961e6b69379987af5a480153b5071