8000 Remove default plugin creation by chaosk · Pull Request #6468 · django-cms/django-cms · GitHub
[go: up one dir, main page]

Skip to content

Remove default plugin creation #6468

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

Conversation

chaosk
Copy link
Contributor
@chaosk chaosk commented Aug 2, 2018

Summary

Based on #6460. Removed logic to create default plugins when placeholder is empty.

Documentation checklist

  • I have updated CHANGELOG.txt if appropriate
  • I have updated the release notes document if appropriate, with:
    • general notes
    • bug-fixes
    • improvements/new features
    • backwards-incompatible changes
    • required upgrade steps
    • names of contributors
  • I have updated other documentation
  • I have added my name to the AUTHORS file
  • This PR's documentation has been approved by Daniele Procida

@coveralls
Copy link
coveralls commented Aug 2, 2018

Coverage Status

Coverage remained the same at 78.189% when pulling e9c4ecb on chaosk:feature/remove-default-plugin-creation into 71a68e1 on divio:release/4.0.x.

@czpython czpython self-assigned this Aug 6, 2018
@czpython czpython added this to the 4.0.0 milestone Aug 6, 2018
@@ -197,14 +197,6 @@ Example::
'extra_context': {"width":640},
'name': gettext("Content"),
'language_fallback': True,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this

@@ -499,8 +492,8 @@ def test_get_placeholder_conf(self):
returned = get_placeholder_conf('excluded_plugins', 'main', 'layout/other.html')
self.assertEqual(returned, TEST_CONF['layout/other.html main']['excluded_plugins'])
# test grandparent inherited value
returned = get_placeholder_conf('default_plugins', 'main', 'layout/other.html')
self.assertEqual(returned, TEST_CONF['main']['default_plugins'])
returned = get_placeholder_conf('language_fallback', 'main', 'layout/other.html')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

language_fallback was removed

@chaosk chaosk force-pushed the feature/remove-default-plugin-creation branch from db3ee71 to e9c4ecb Compare August 6, 2018 10:28
@czpython czpython merged commit eef5cbb into django-cms:release/4.0.x Aug 6, 2018
fsbraun added a commit to fsbraun/django-cms that referenced this pull request May 14, 2024
fsbraun added a commit to fsbraun/django-cms that referenced this pull request May 14, 2024
fsbraun added a commit that referenced this pull request May 22, 2024
* Revert "Removed default plugin creation for placeholders (#6468)"

This reverts commit eef5cbb.

* Fix tests

* Fix codespell error

---------

Co-authored-by: Vinit Kumar <vinit.kumar@kidskonnect.nl>
vinitkumar pushed a commit that referenced this pull request May 24, 2024
…nt) (#7928)

* Revert "Removed default plugin creation for placeholders (#6468)"

This reverts commit eef5cbb.

* Fix: Only change structure board endpoint in edit mode

* Fix typo

* Revert "Revert "Removed default plugin creation for placeholders (#6468)""

This reverts commit 71f0d7f.

* fix: Correct patch

* Update cms.structureboard.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0