-
Notifications
You must be signed in to change notification settings - Fork 25
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
Refactor and add new functionality #74
Conversation
@kpodemski Ready to go :-) |
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 like this :)
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.
WH
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.
Hello @Hlavtox ,
I see that the version has changed to v3.0.0. But I don't see the bump.
In my BO, I still have v2.0.3.
Could you take a look please ?
Thanks :)
@florine2623 Try two things
Go back and you should see 3.0.0 :-) |
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.
Hello @Hlavtox ,
Thanks for the PR !
Tested the following :
- bump to v3.0.0
- tested all settings options, checked on FO the result
LGTM ✅
Changes
in_path
boolean in every category.What it enables to do
So now with just a few template modifications, throwing
{if $node.in_path}
here and there, you can render the category tree opened.