Adds "popular" charms to sidebar.

Adds "popular" charms to sidebar.

-Another block in sidebar creates the "popular" charm container.
-"Popular" charms also added to cache.
-"Popular" block added to sidebar template.
-Added "popular" to sidebar json test data, and re-pretty-printed it.
-Updated test for the additional charm in cache; although "popular" has two
charms, one of them (cassandra) is also in our "new" category.

https://code.launchpad.net/~jcsackett/juju-gui/add-new-to-sidebar/+merge/158397

(do not edit description out of merge proposal)

[j.c.sackett, 2013-04-11 15:14:04]

Please take a look.

[rharding, 2013-04-11 15:33:45]

LGTM with comments below.

https://codereview.appspot.com/8561048/diff/1/test-server.sh
File test-server.sh (right):

https://codereview.appspot.com/8561048/diff/1/test-server.sh#newcode8
test-server.sh:8: xdg-open http://0.0.0.0:8084/test/index.html
please back out again

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.json
File test/data/sidebar_editorial.json (right):

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.jso...
test/data/sidebar_editorial.json:3: "result":{
removing the spaces after the : and at the EOL is actually anti-json. Maybe run
this through an online prettifier or a python json.loads() json.dumps again.

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.jso...
test/data/sidebar_editorial.json:405: "is_popular":false,
should be true

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.jso...
test/data/sidebar_editorial.json:405: "is_popular":false,
should be true.

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.jso...
test/data/sidebar_editorial.json:620: "is_popular":false,
like the server side, this should be true.

[j.c.sackett, 2013-04-11 15:52:26]

https://codereview.appspot.com/8561048/diff/1/test-server.sh
File test-server.sh (right):

https://codereview.appspot.com/8561048/diff/1/test-server.sh#newcode8
test-server.sh:8: xdg-open http://0.0.0.0:8084/test/index.html
On 2013/04/11 15:33:45, rharding wrote:
> please back out again

Argh, right.

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.json
File test/data/sidebar_editorial.json (right):

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.jso...
test/data/sidebar_editorial.json:3: "result":{
On 2013/04/11 15:33:45, rharding wrote:
> removing the spaces after the : and at the EOL is actually anti-json. Maybe
run
> this through an online prettifier or a python json.loads() json.dumps again.

Surprisingly, this is generated by json.dumps with the recommended "prettyprint"
options.

I'll fix it.

https://codereview.appspot.com/8561048/diff/1/test/data/sidebar_editorial.jso...
test/data/sidebar_editorial.json:405: "is_popular":false,
On 2013/04/11 15:33:45, rharding wrote:
> should be true
Ack.

[j.c.sackett, 2013-04-11 19:07:28]

Please take a look.

[jeff.pihach, 2013-04-11 19:11:15]

LGTM thanks!

[j.c.sackett, 2013-04-11 19:25:16]

*** Submitted:

Adds "popular" charms to sidebar.

-Another block in sidebar creates the "popular" charm container.
-"Popular" charms also added to cache.
-"Popular" block added to sidebar template.
-Added "popular" to sidebar json test data, and re-pretty-printed it.
-Updated test for the additional charm in cache; although "popular" has two
charms, one of them (cassandra) is also in our "new" category.

R=rharding, jeff.pihach
CC=
https://codereview.appspot.com/8561048