[ckan-dev] 2375-demo-theme code review

Wed Sep 5 15:58:50 UTC 2012

Work assigned to people see below

On 5 September 2012 13:17, Toby Dacre <toby.okfn at gmail.com> wrote:

> Hi,
>
> To help with the review I've split it into parts please review the parts
> including documentation if relevant.  Email/github or trac any problems and
> I'll do what I can to help/fix etc or make patches and send them for me to
> do a quick review.  The main files involved are listed here but also good
> to check related code ie if helper function is new have a quick look at it
> etc
>
> Can people either volunteer for these or else we'll dish them out at this
> afternoons meeting  The auth/related stuff was done by Ross so he shouldn't
> review them
>
> Translations: sean
> ------------------
> includes setup.py,
> lib/extract.py
> lib/jinja_extensions.py trans tag changes CkanInternationalizationExtension
> lib/cli.py TranslationsCommand
>
    ckan/lib/formatters.py

>
> Jinja2 stuff ian
> ---------------
> lib/jinja_extensions
> lib/render.py
> lib/base.py
> the templates?
>
> admin david
> ---------
>  ckan/controllers/admin.py
>
> authz sean
> --------
> ckan/authz.py
>
> package david
> ------------
>  ckan/controllers/package.py
>
> related toby
> ---------
>  ckan/controllers/related.py
>
> home david
> -------
>  ckan/controllers/home.py
>
> other controllers (smaller changes) david
> ---------------------
> ckan/controllers/api.py
>  ckan/controllers/error.py
>  ckan/controllers/group.py
>  ckan/controllers/user.py
>  ckan/controllers/util.py
>
>
> helper functions
> -----------------------
>
>  ckan/lib/helpers.py   ian
>
> globals david
> ----------
> ckan/lib/app_globals.py
>
> config changes ian
> ---------------------
>  ckan/config/deployment.ini_tmpl
>  ckan/config/environment.py
>  ckan/config/middleware.py
>  ckan/config/routing.py
>
> fanstatic ian
> ------------
> ckan/fanstatic*.py
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.okfn.org/pipermail/ckan-dev/attachments/20120905/06d6d444/attachment-0001.html>