-
Notifications
You must be signed in to change notification settings - Fork 6
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
Plone6 #603
Conversation
1c7b885
to
5dea889
Compare
5ba2038
to
1f0281b
Compare
8a1b1e6
to
de26211
Compare
483ad59
to
bd0ed0a
Compare
@@ -660,7 +659,6 @@ def fixed_priority(self, value): | |||
self.default_priority = value | |||
|
|||
@property | |||
@memoize |
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.
I'm fine with using memoize sparsely and keep this change, but what problem does this solve?
Seems unrelated to the Plone 6 changes here.
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 broke some test for some reason I do not remember why
No description provided.