-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
fix: Allow lazy wizard initialization #8267
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
Reviewer's GuideThis PR removes direct use of the deprecated wizard_pool API by replacing it with the new get_entry helper and updates the deprecation warning to target the upcoming CMS 5.0 release. Class diagram for Wizard entry retrieval updateclassDiagram
class WizardCreateView {
+get_selected_entry()
}
class wizard_pool {
+get_entry()
}
class get_entry
WizardCreateView --> get_entry : uses
%% wizard_pool is no longer used directly by WizardCreateView
Class diagram for updated deprecation warning in wizard_poolclassDiagram
class wizard_pool {
+register(entry)
}
class RemovedInDjangoCMS50Warning
wizard_pool ..> RemovedInDjangoCMS50Warning : uses in warning
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
👋 Hi there! Please remember to MERGE COMMIT pull requests from Do not SQUASH commits to preserve history for the changelog. |
👋 Hi there! Please remember to MERGE COMMIT pull requests from Do not SQUASH commits to preserve history for the changelog. |
…views (#8067) * Fix: GrouperModelAdmin raised an AttributeError if used outside the admin app * Fixed: Page.__str__ returns page title with admin_manager * feat: Format `Page.__str__` as "My title (/path/to/page/)" * Update tests for new str method of Page * Simplify str --------- Co-authored-by: Github Release Action <info@django-cms.org> Co-authored-by: Vinit Kumar <mail@vinitkumar.me>
…views (#8067) * Fix: GrouperModelAdmin raised an AttributeError if used outside the admin app * Fixed: Page.__str__ returns page title with admin_manager * feat: Format `Page.__str__` as "My title (/path/to/page/)" * Update tests for new str method of Page * Simplify str --------- Co-authored-by: Github Release Action <info@django-cms.org> Co-authored-by: Vinit Kumar <mail@vinitkumar.me>
👋 Hi there! Please remember to MERGE COMMIT pull requests from Do not SQUASH commits to preserve history for the changelog. |
👋 Hi there! Please remember to MERGE COMMIT pull requests from Do not SQUASH commits to preserve history for the changelog. |
👋 Hi there! Please remember to MERGE COMMIT pull requests from Do not SQUASH commits to preserve history for the changelog. |
👋 Hi there! Please remember to MERGE COMMIT pull requests from Do not SQUASH commits to preserve history for the changelog. |
👋 Hi there! Please remember to MERGE COMMIT pull requests from Do not SQUASH commits to preserve history for the changelog. |
Description
Backport of #8265
-->
Related resources
Checklist
main
Summary by Sourcery
Replace deprecated wizard_pool usage with get_entry and adjust its deprecation version.
Enhancements: