Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add plugin module/name settings #287

Merged
merged 1 commit into from
Nov 30, 2015

Conversation

suquant
Copy link
Contributor

@suquant suquant commented Nov 27, 2015

No description provided.

@FinalAngel
Copy link
Member

thank you very much for the PR @suquant
@yakky is this valid for you? though tests are failing...

@yakky
Copy link
Member

yakky commented Nov 30, 2015

@suquant nice! could you fix the isort errors?
@FinalAngel wondering if we might want a generic way to do this in the CMS

@FinalAngel
Copy link
Member

@yakky how would we do this generically? The plugin should probably set this in general. Less logic at cms :D

@yakky
Copy link
Member

yakky commented Nov 30, 2015

@FinalAngel a setting in the CMS that allows to override module and name for plugin classes. This is not the place to discuss this, though. Will post on ML

@yakky
Copy link
Member

yakky commented Nov 30, 2015

@FinalAngel merging this
@suquant thanks!

yakky added a commit that referenced this pull request Nov 30, 2015
@yakky yakky merged commit 59224a2 into django-cms:master Nov 30, 2015
@coveralls
Copy link

Coverage Status

Coverage increased (+20.6%) to 83.333% when pulling dcf5e73 on peacedata:addition_settings into ce5ff80 on divio:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants