django-cms · fsbraun · Aug 20, 2024 · Sep 8, 2021 · Sep 15, 2021 · Sep 17, 2021
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
@@ -26,5 +26,6 @@ Use 'x' to check each item: [x] I have ...
 * [ ] I have opened this pull request against ``master``
 * [ ] I have added or modified the tests when changing logic
 * [ ] I have followed [the conventional commits guidelines](https://www.conventionalcommits.org/) to add meaningful information into the changelog
-* [ ] I have read the [contribution guidelines ](https://github.com/django-cms/django-cms/blob/develop/CONTRIBUTING.rst) and I have joined #workgroup-pr-review on
-[Slack](https://www.django-cms.org/slack) to find a “pr review buddy” who is going to review my pull request.
+* [ ] I have read the [contribution guidelines ](https://github.com/django-cms/django-cms/blob/develop/CONTRIBUTING.rst) and I have joined #pr-review on
+[Discord](https://discord-pr-review-channel.django-cms.org/) to find a “pr review buddy” who is
+  going to review my pull request.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -2,8 +2,6 @@ name: CodeCov
 
 on:
   push:
-    branches:
-    - master
   pull_request:
 
 concurrency:

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -8,30 +8,18 @@ ci:
     autoupdate_schedule: monthly
 
 repos:
-  -  repo: https://github.com/asottile/pyupgrade
-     rev: v3.15.2
-     hooks:
-       -  id: pyupgrade
-          args: ["--py38-plus"]
-
-  - repo: https://github.com/adamchainz/django-upgrade
-    rev: '1.16.0'
+  - repo: https://github.com/asottile/pyupgrade
+    rev: v2.31.0
     hooks:
-      - id: django-upgrade
-        args: [--target-version, "3.2"]
+      - id: pyupgrade
+        args: ["--py38-plus"]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.4.3"
+    rev: v0.4.4
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
 
-  - repo: https://github.com/astral-sh/ruff-pre-commit
-    # Ruff version.
-    rev: v0.4.3
-    hooks:
-      - id: ruff-format
-
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.6.0
     hooks:
@@ -43,8 +31,3 @@ repos:
       - id: check-toml
       - id: end-of-file-fixer
       - id: trailing-whitespace
-
-#  - repo: https://github.com/pre-commit/mirrors-mypy
-#    rev: v1.3.0
-#    hooks:
-#      - id: mypy
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -5,8 +5,64 @@ Changelog
 Unreleased
 ==========
 
-* Drop support for Django < 3.2
-* Drop support for Python < 3.9
+* feat: Universal support for django CMS 3.11 and 4.x
+
+
+4.1.0 (2024-05-16)
+==================
+
+* feat: Added sites support for Snippets
+* add support for python 3.10
+* add support for django >= 4.2
+* drop support for django < 3.2
+* drop support python < 3.8
+
+
+4.0.1.dev2 (2022-11-15)
+=======================
+
+* feat: Enable add button to create a snippet when adding a SnippetPlugin
+
+
+4.0.1.dev1 (2022-05-10)
+=======================
+
+* Python 3.8, 3.9 support added
+* Django 3.0, 3.1 and 3.2 support added
+* Python 3.5 and 3.6 support removed
+* Django 1.11 support removed
+* port-feat: pre-commit config added from the v3 workstream
+* fix: Added test coverage to admin preview view
+
+
+4.0.0.dev4 (2022-02-03)
+=======================
+
+* feat: Preview icon renders form in read only mode
+
+
+4.0.0.dev3 (2022-01-11)
+=======================
+
+* fix: Snippet plugin added to a page now displays name instead of ID
+* fix: Slug field on list display for admin should only be displayed when versioning is not available
+* fix: Removed unused contents within templates, reducing the clutter within version compare views. Previously this was causing a lot of junk to be included in the version comparison, this will now be reduced.
+
+
+4.0.0.dev2 (2021-12-22)
+=======================
+
+* fix: Removed tight django-treebeard restriction added when 4.5.0 contained breaking changes. The core CMS and django-treebeard have since been patched to resolve the issue.
+
+
+4.0.0.dev1 (2021-12-14)
+=======================
+
+* feat: Exposed the setting DJANGOCMS_SNIPPET_VERSIONING_MIGRATION_USER_ID as an environment variable for the Divio addon
+* fix: Error when rendering a Draft Snippet plugin on a Published page
+* fix: Publish snippets by default as they were already in that state pre-versioning and cleanup unnecessary migration files before release!
+* feat: djangocms-versioning support added, including model restructure and configuration
+* feat: django-cms v4.0.x support added
 
 3.1.1
 =====

diff --git a/README.rst b/README.rst
@@ -2,7 +2,7 @@
 django CMS Snippet
 ==================
 
-|pypi| |coverage| |python| |django| |djangocms|
+|pypi| |coverage| |python| |django| |djangocms| |django-cms4|
 
 
 **django CMS Snippet** provides a plugin for `django CMS <http://django-cms.org>`_
@@ -89,6 +89,12 @@ please set ``DJANGOCMS_SNIPPET_CACHE`` to ``False`` in your settings::
 
     DJANGOCMS_SNIPPET_CACHE = False # default value is False
 
+Migration 0010 requires the use of a user in order to create versions for existing snippets (if djangocms_versioning is installed and enabled),
+a user can be chosen with the setting ``DJANGOCMS_SNIPPET_VERSIONING_MIGRATION_USER_ID``, the default is 1.
+This setting is also exposed as an Environment variable for Divio projects using the Divio addon.
+
+    DJANGOCMS_SNIPPET_VERSIONING_MIGRATION_USER_ID = 2 # Will use user with id: 2
+
 Template tag
 ------------
 
@@ -125,9 +131,11 @@ You can run tests by executing::
     :target: http://badge.fury.io/py/djangocms-snippet
 .. |coverage| image:: https://codecov.io/gh/django-cms/djangocms-snippet/branch/master/graph/badge.svg
     :target: https://codecov.io/gh/django-cms/djangocms-snippet
-.. |python| image:: https://img.shields.io/badge/python-3.5+-blue.svg
+.. |python| image:: https://img.shields.io/badge/python-3.9+-blue.svg
     :target: https://pypi.org/project/djangocms-snippet/
-.. |django| image:: https://img.shields.io/badge/django-2.2,%203.0,%203.1-blue.svg
+.. |django| image:: https://img.shields.io/badge/django-3.2+-blue.svg
     :target: https://www.djangoproject.com/
 .. |djangocms| image:: https://img.shields.io/badge/django%20CMS-3.7%2B-blue.svg
     :target: https://www.django-cms.org/
+.. |djangocms4| image:: https://img.shields.io/badge/django%20CMS-4-blue.svg
+    :target: https://www.django-cms.org/
diff --git a/aldryn_config.py b/aldryn_config.py
@@ -1,3 +1,5 @@
+from functools import partial
+
 from aldryn_client import forms
 
 
@@ -17,10 +19,22 @@ class Form(forms.BaseForm):
     )
 
     def to_settings(self, data, settings):
-        if data["editor_theme"]:
-            settings["DJANGOCMS_SNIPPET_THEME"] = data["editor_theme"]
-        if data["editor_mode"]:
-            settings["DJANGOCMS_SNIPPET_MODE"] = data["editor_mode"]
-        if data["enable_search"]:
-            settings["DJANGOCMS_SNIPPET_SEARCH"] = data["enable_search"]
+        from aldryn_addons.utils import djsenv
+
+        env = partial(djsenv, settings=settings)
+
+        # Get a migration user if the env setting has been added
+        migration_user_id = env(
+            'DJANGOCMS_SNIPPET_VERSIONING_MIGRATION_USER_ID',
+            default=False
+        )
+        if migration_user_id:
+            settings['DJANGOCMS_SNIPPET_VERSIONING_MIGRATION_USER_ID'] = int(migration_user_id)
+
+        if data['editor_theme']:
+            settings['DJANGOCMS_SNIPPET_THEME'] = data['editor_theme']
+        if data['editor_mode']:
+            settings['DJANGOCMS_SNIPPET_MODE'] = data['editor_mode']
+        if data['enable_search']:
+            settings['DJANGOCMS_SNIPPET_SEARCH'] = data['enable_search']
         return settings
diff --git a/pyproject.toml b/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "djangocms-snippet"
-version = "3.1.1"
+dynamic = ["version"]
 authors = [
   {name = "Divio AG", email = "[email protected]"},
 ]
@@ -26,9 +26,9 @@ classifiers=[
     "Operating System :: OS Independent",
     "Programming Language :: Python",
     "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.8",
     "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
     "Framework :: Django",
     "Framework :: Django :: 3.2",
     "Framework :: Django :: 4.2",
@@ -37,6 +37,8 @@ classifiers=[
     "Framework :: Django CMS :: 3.9",
     "Framework :: Django CMS :: 3.10",
     "Framework :: Django CMS :: 3.11",
+    "Framework :: Django CMS :: 4.0",
+    "Framework :: Django CMS :: 4.1",
     "Topic :: Internet :: WWW/HTTP",
     "Topic :: Internet :: WWW/HTTP :: Dynamic Content",
     "Topic :: Software Development",
@@ -91,8 +93,8 @@ allow_untyped_defs = true
 
 [tool.ruff]
 # https://beta.ruff.rs/docs/configuration/
-line-length = 79
-select = [
+line-length = 120
+lint.select = [
   "E",    # pycodestyle errors
   "W",    # pycodestyle warnings
   "F",    # pyflakes
@@ -126,17 +128,20 @@ exclude = [
   "venv",
 ]
 
-ignore = [
+lint.ignore = [
   "E501",  # line-too-long
   "W191",  # tab-indentation
 ]
 
-[tool.ruff.per-file-ignores]
+[tool.ruff.lint.per-file-ignores]
 "__init__.py" = [
   "F401"  # unused-import
 ]
+"test_plugins.py" = [
+  "FBT003",  # Boolean positional value in function call
+]
 
-[tool.ruff.isort]
+[tool.ruff.lint.isort]
 combine-as-imports = true
 known-first-party = [
   "djangocms_snippet",

diff --git a/requirements.in b/requirements.in
@@ -1,6 +1,6 @@
 bump2version
 django-cms>=3.7
-django-treebeard>=4.3,<4.5
+django-treebeard
 pip-tools
 pre-commit
 wheel
diff --git a/requirements.txt b/requirements.txt
@@ -1,72 +1,85 @@
 #
-# This file is autogenerated by pip-compile with python 3.9
-# To update, run:
+# This file is autogenerated by pip-compile with Python 3.9
+# by the following command:
 #
-#    pip-compile
+#    pip-compile --output-file=requirements.txt requirements.in
 #
-asgiref==3.6.0
+asgiref==3.8.1
     # via django
+build==1.2.1
+    # via pip-tools
 bump2version==1.0.1
     # via -r requirements.in
-cfgv==3.3.1
+cfgv==3.4.0
     # via pre-commit
-click==8.0.3
+click==8.1.7
     # via pip-tools
-distlib==0.3.4
+distlib==0.3.8
     # via virtualenv
-django==4.1.10
+django==4.2.13
     # via
     #   django-classy-tags
     #   django-cms
     #   django-formtools
     #   django-sekizai
     #   django-treebeard
-django-classy-tags==3.0.0
+    #   djangocms-admin-style
+django-classy-tags==4.1.0
     # via
     #   django-cms
     #   django-sekizai
-django-cms==3.9.0
+django-cms==4.1.1
     # via -r requirements.in
-django-formtools==2.3
+django-formtools==2.5.1
     # via django-cms
-django-sekizai==3.0.0
+django-sekizai==4.1.0
     # via django-cms
-django-treebeard==4.4
+django-treebeard==4.7.1
     # via
     #   -r requirements.in
     #   django-cms
-djangocms-admin-style==3.0.0
+djangocms-admin-style==3.3.1
     # via django-cms
-filelock==3.4.2
+filelock==3.14.0
     # via virtualenv
-identify==2.4.4
+identify==2.5.36
     # via pre-commit
-nodeenv==1.6.0
+importlib-metadata==7.1.0
+    # via build
+nodeenv==1.8.0
     # via pre-commit
-pep517==0.12.0
-    # via pip-tools
-pip-tools==6.4.0
+packaging==24.0
+    # via
+    #   build
+    #   django-cms
+pip-tools==7.4.1
     # via -r requirements.in
-platformdirs==2.4.1
+platformdirs==4.2.2
     # via virtualenv
-pre-commit==2.17.0
+pre-commit==3.7.1
     # via -r requirements.in
-pyyaml==6.0
+pyproject-hooks==1.1.0
+    # via
+    #   build
+    #   pip-tools
+pyyaml==6.0.1
     # via pre-commit
-six==1.16.0
-    # via virtualenv
 sqlparse==0.5.0
     # via django
-toml==0.10.2
-    # via pre-commit
-tomli==2.0.0
-    # via pep517
-virtualenv==20.13.0
+tomli==2.0.1
+    # via
+    #   build
+    #   pip-tools
+typing-extensions==4.11.0
+    # via asgiref
+virtualenv==20.26.2
     # via pre-commit
-wheel==0.38.1
+wheel==0.43.0
     # via
     #   -r requirements.in
     #   pip-tools
+zipp==3.18.2
+    # via importlib-metadata
 
 # The following packages are considered to be unsafe in a requirements file:
 # pip
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,8 +2,6 @@ name: CodeCov @@
     on:
       push:
-        branches:
-        - master
       pull_request:
     concurrency:
@@ Expand Down @@