Skip to content

Conversation

fsbraun
Copy link
Member

@fsbraun fsbraun commented May 8, 2025

Description

At testing I had to fix two more bugs:

  • Plugins that require a parent were not shown in the plugin selector box (omg) - a missing parameter at the plugin_pool.registered_plugins property.
  • The plugin pool accessed the page cache without need at project startup

This is read to go from my side.

Related resources

  • #...
  • #...

Checklist

  • I have opened this pull request against main
  • I have added or modified the tests when changing logic
  • I have followed the conventional commits guidelines to add meaningful information into the changelog
  • I have read the contribution guidelines and I have joined the channel #pr-reviews on our Discord Server to find a “pr review buddy” who is going to review my pull request.

Summary by Sourcery

Merge back django CMS 5.0.0 build, updating version and font references from 5.0.0a1 to 5.0.0

New Features:

  • Updated to final 5.0.0 version

Chores:

  • Updated font and icon references from 5.0.0a1 to 5.0.0
  • Updated static CSS and JavaScript files to 5.0.0 version

Summary by Sourcery

Update project assets and configuration to reflect the django CMS 5.0.0 release, including static files, versioning, and compatibility updates.

Enhancements:

  • Update static asset references and versioning from 5.0.0a1 to 5.0.0.

Build:

  • Add new CSS files for 5.0.0 and remove obsolete 5.0.0a1 assets.

CI:

  • Update CI workflow to test against Python 3.13 and Django 5.2.

Documentation:

  • Update documentation and upgrade guides for the 5.0.0 release.

Tests:

  • Adjust tests to align with plugin pool and cache changes.

Copy link
Contributor

sourcery-ai bot commented May 8, 2025

Reviewer's Guide

This PR merges back the django CMS 5.0.0 build, finalizing the version upgrade from 5.0.0a1 to 5.0.0, updating static assets, font references, and code to reflect the final release, and cleaning up legacy or debug code.

File-Level Changes

Change Details Files
Update version and static asset references from 5.0.0a1 to 5.0.0.
  • Bump version to 5.0.0.
  • Update font-face and static asset paths in SCSS and CSS files to use 5.0.0.
  • Replace static CSS/JS files for 5.0.0a1 with 5.0.0 versions.
  • Remove 5.0.0a1 static files and add 5.0.0 equivalents.
cms/__init__.py
cms/static/cms/sass/components/_iconography.scss
cms/static/cms/css/5.0.0/cms.admin.css
cms/static/cms/css/5.0.0/cms.base.css
cms/static/cms/css/5.0.0/cms.welcome.css
cms/static/cms/js/dist/5.0.0a1/bundle.admin.base.min.js
cms/static/cms/css/5.0.0a1/cms.admin.css
cms/static/cms/css/5.0.0a1/cms.base.css
cms/static/cms/css/5.0.0a1/cms.pagetree.css
cms/static/cms/css/5.0.0a1/cms.welcome.css
cms/static/cms/js/dist/5.0.0a1/bundle.toolbar.min.js
Update plugin pool and cache handling logic for plugin discovery and cache invalidation.
  • Remove legacy cache invalidation from plugin discovery.
  • Update registered_plugins property to use new plugin retrieval logic.
  • Simplify test for cache invalidation and plugin pool reset.
cms/plugin_pool.py
cms/tests/test_cache.py
Remove debug and print statements from tests.
  • Remove print statements from test_templatetags.py.
cms/tests/test_templatetags.py
Update CI workflow and test matrix for new Python and Django versions.
  • Add Django 5.2 and Python 3.13 to test matrix in GitHub Actions workflow.
.github/workflows/test_startcmsproject.yml
Documentation and translation updates for 5.0.0 release.
  • Update changelog and upgrade documentation for 5.0.0.
  • Add/adjust upgrade docs for previous 4.1.x releases.
  • Update translation files.
CHANGELOG.rst
docs/upgrade/5.0.0.rst
docs/upgrade/index.rst
docs/upgrade/version.rst.template
docs/upgrade/4.1.2.rst
docs/upgrade/4.1.3.rst
docs/upgrade/4.1.4.rst
docs/upgrade/4.1.5.rst
docs/upgrade/4.1.6.rst
cms/locale/en/LC_MESSAGES/django.po
cms/locale/zh_CN/LC_MESSAGES/django.po

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@fsbraun fsbraun marked this pull request as ready for review May 11, 2025 07:29
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @fsbraun - I've reviewed your changes and they look great!

Here's what I looked at during the review
  • 🟡 General issues: 17 issues found
  • 🟢 Security: all looks good
  • 🟡 Testing: 1 issue found
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
@vinitkumar vinitkumar merged commit 0f633fc into develop-4 May 11, 2025
66 checks passed
@github-actions github-actions bot deleted the release/build branch May 12, 2025 09:38
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.

2 participants