-
-
Notifications
You must be signed in to change notification settings - Fork 182
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
fix navtree deprecations #3756
fix navtree deprecations #3756
Conversation
@jensens thanks for creating this Pull Request and helping to improve Plone! TL;DR: Finish pushing changes, pass all other checks, then paste a comment:
To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass, but it takes 30-60 min. Other CI checks are usually much faster and the Plone Jenkins resources are limited, so when done pushing changes and all other checks pass either start all Jenkins PR jobs yourself, or simply add the comment above in this PR to start all the jobs automatically. Happy hacking! |
3b15100
to
ea20ca7
Compare
ea20ca7
to
e9c1434
Compare
@jenkins-plone-org please run jobs |
Can you please split this into one commit with black/isort and one with the interesting changes? That is much easier to review. Now it is not clear at all what the actually interesting changes are. |
Well, its using interfaces and utilities from |
Products/CMFPlone/browser/navtree.py
Outdated
newNode["review_state"] = getattr(item, "review_state", None) | ||
newNode["Description"] = getattr(item, "Description", None) | ||
newNode["show_children"] = showChildren | ||
newNode["no_display"] = False # We sort this out with the nodeFilter | ||
# BBB getRemoteUrl and link_remote are deprecated, remove in Plone 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😆 we are a bit late at deprecating these BBB
comment 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice if you can structure your commits with automatic changes (either first or last) and then the actual changes, that would make reviewing much easier ✨
@jenkins-plone-org please run jobs |
@jenkins-plone-org please run jobs |
12782a9
to
99a2bab
Compare
@jenkins-plone-org please run jobs |
No description provided.