lp:~fabricematrat/charmworld/redirect-charm

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:charmworld at revision 520

Brad Crittenden (community): Approve (code) on 2015-02-09

Juju Gui Bot: Approve (continuous-integration) on 2015-02-09

j.c.sackett (community): Approve on 2015-02-09

Diff: 470 lines (+88/-225)

3 files modified

charmworld/views/charms.py (+42/-21)
charmworld/views/tests/test_charms.py (+42/-203)
default.ini (+4/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

525. By Fabrice Matrat on 2015-02-09

Lint error

524. By Fabrice Matrat on 2015-02-09

Remove useless code, simplification, typos

523. By Fabrice Matrat on 2015-02-09

Added assert in tests

522. By Fabrice Matrat on 2015-02-09

Redirect for series added

521. By Fabrice Matrat on 2015-02-09

first redirect for charmworld to jujucharms.com including personal, distro, promulgated charms

520. By Fabrice Matrat on 2015-02-09

first redirect for charmworld to jujucharms.com

519. By Brad Crittenden on 2015-01-13

Update for rebuilding ES in a production or production-like environment.
R=rharding.

Approved by Juju Gui Bot, Richard Harding.

518. By Brad Crittenden on 2014-10-16

Add 'start' parameter to API search.

The start is the zero-based index into the search results. It can be used
in conjunction with limit.

https://codereview.appspot.com/158010043/

R=jcsackett. Fixes: https://bugs.launchpad.net/bugs/1379397.

Approved by Juju Gui Bot.

517. By j.c.sackett on 2014-09-05

Update charmworld to use charmworldlib 0.4.2.

Approved by Juju Gui Bot, j.c.sackett.

516. By Brad Crittenden on 2014-07-16

Make ngram searches work.

After the ngram search was deployed it was discovered to only work if 1) using
the API and 2) had autocomplete=true.

Inspecting the code showed the intent for ngram to be used in other
situations, and there was not documented rationale for only using it when
doing autocomplete.

The problem was simply in the query for non-autocomplete the field for the
ngrams was prefixed with 'data.' when it should not have been.

The ngrams fields were moved out of the charm and bundle fields and passed as
separate fields, treated like 'provides' and 'requires', to avoid the
prefixing.

As a result of this change, the search on the manage.jujucharms.com page now
uses ngrams too.

https://codereview.appspot.com/117810044/

R=makyo. Fixes: https://bugs.launchpad.net/bugs/1220909.

Approved by Juju Gui Bot, Matthew Scott.