-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add a module global COLLECTIONISH_TARGETS #154
Conversation
c7e5d09
to
72413c5
Compare
95d3e03
to
e237998
Compare
I think its because of the pinned |
6d161c1
to
f48c371
Compare
f48c371
to
b8b4fb8
Compare
All Systems Red. (btw a good scifi read by Martha Wells) sigh here ;-) |
strangely also the merge commit from yesterday turned red although the PR was green 🧐 ... somehow the |
@@ -213,6 +217,9 @@ def get_filter_items( | |||
or filter_value in value_blacklist | |||
): | |||
continue | |||
if isinstance(filter_value, int): |
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.
This breaks boolean filter_value
because isinstance(<False or True>, int) == True
and therefore boolean values get converted to strings
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.
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.
yay.
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 fixed that ... and portlets too with the new reverse
setting
5640216
to
22112cc
Compare
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.
LGTM
to register other tiles with collections than plone.app.standardtiles.contentlisting
Also: Support Plone 6