Re: Review Request 23329: remove embedded jquery

2014-10-23 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review58218 --- This patch does not apply cleanly on master (53f4e73), do you need

Re: Review Request 23329: remove embedded jquery

2014-09-29 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review54897 --- Can this be discarded? - Kevin Sweeney On July 9, 2014, 2:50

Re: Review Request 23329: remove embedded jquery

2014-07-30 Thread David Robinson
On July 30, 2014, 4:25 a.m., David McLaughlin wrote: Not sure I understand the motivation here. It's generally best to host your own static assets. Users typically debug their Aurora jobs by looking at individual instances, which usually involves accessing the web interfaces (provided by

Re: Review Request 23329: remove embedded jquery

2014-07-30 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review49153 --- ./src/main/python/apache/thermos/observer/http/assets/jquery*

Re: Review Request 23329: remove embedded jquery

2014-07-29 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review49045 --- Reviewers - ping? - Bill Farner On July 9, 2014, 9:50 p.m.,

Re: Review Request 23329: remove embedded jquery

2014-07-29 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review49084 --- Not sure I understand the motivation here. It's generally best to

Re: Review Request 23329: remove embedded jquery

2014-07-09 Thread David Robinson
On July 8, 2014, 1:18 a.m., Kevin Sweeney wrote: src/main/python/apache/thermos/observer/http/templates/filebrowse.tpl, line 62 https://reviews.apache.org/r/23329/diff/1/?file=625207#file625207line62 -1, this is pretty much unacceptable from a security standpoint - data

Re: Review Request 23329: remove embedded jquery

2014-07-09 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/ --- (Updated July 9, 2014, 9:50 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 23329: remove embedded jquery

2014-07-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review47428 ---