django-blog-zinnia / cmsplugin-zinnia

Bridge between django-blog-zinnia and django-cms
BSD 3-Clause "New" or "Revised" License
53 stars 46 forks source link

appending zinnia in the application namespace #32

Closed jonhairston closed 9 years ago

jonhairston commented 9 years ago

Hey,

After some time spent wondering why django-cms wasn't allowing me to add 'zinnia' to the application namespace in the page setup in the admin, I came across the option of adding the app_name = 'zinnia' field to the ZinniaAppHook, which sets the application namespace in the cms_app.py.

Like this:

class ZinniaApphook(CMSApp):
    """
    Zinnia's Apphook
    """
    name = _('Zinnia Blog')
    urls = APP_URLS
    menus = app_menus
    app_name = 'zinnia' ### This line here ###

I was wondering if you guys wouldn't mind adding that in there. It could save others at least a few mins.

Fantomas42 commented 9 years ago

Hello,

when I was looking at the documentation and the code, the app_name attribut was not available or did not work as expected. Maybe it's time to use it.

Tests are needed.