-
-
Notifications
You must be signed in to change notification settings - Fork 223
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
FEATURE: Timeable Node Visibility #4817
Conversation
a659875
to
087f440
Compare
087f440
to
98d151e
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.
THANK you so much! I just had a quick look over the files (except the behavioural tests ^^)
Neos.TimeableNodeVisibility/Classes/Command/TimeableNodeVisibilityCommandController.php
Show resolved
Hide resolved
Neos.TimeableNodeVisibility/Classes/Command/TimeableNodeVisibilityCommandController.php
Show resolved
Hide resolved
Neos.TimeableNodeVisibility/Classes/Domain/HandlingResultSet.php
Outdated
Show resolved
Hide resolved
Neos.TimeableNodeVisibility/Classes/Service/TimeableNodeVisibilityService.php
Show resolved
Hide resolved
Neos.TimeableNodeVisibility/Classes/Service/TimeableNodeVisibilityService.php
Outdated
Show resolved
Hide resolved
Neos.TimeableNodeVisibility/Classes/Service/TimeableNodeVisibilityService.php
Outdated
Show resolved
Hide resolved
Neos.TimeableNodeVisibility/Classes/Service/TimeableNodeVisibilityService.php
Outdated
Show resolved
Hide resolved
Neos.TimeableNodeVisibility/Classes/Command/TimeableNodeVisibilityCommandController.php
Outdated
Show resolved
Hide resolved
Looks pretty cool, thanks for taking care! Left some comments, but I think this is nicely underway, I agree with concerns regarding performance but don't have amazing ideas right now, apart from, should/could this be a projection. |
Neos.TimeableNodeVisibility/Resources/Private/Translations/en/NodeTypes/Timeable.xlf
Show resolved
Hide resolved
Thanks for this feature. I just played around with a little and i can say that it works :D. At first i had to update your branch a little and to ease installation i pushed a new temporary branch to neos so everyone can add this in their root composer json to experiment: (i actually have no clue how you got it running *g)
A few things to be noted / discussed:
|
I prepared the split: |
Follow up:
|
Re-Implements the already know feature for hiding and/or showing content nodes on a specific point on time.
What has changed in comparision to the old implementation:
./flow timeablenodevisibility:execute
)neos/timeable-node-visibility
) and not part of the Neos minimum installation anymore. So you are able to decide, if you want to enable this feature or not by adding it manually to your dependencies.Renamed the properties:
_hiddenBeforeDateTime
=>enableAfterDateTime
_hiddenAfterDateTime
=>disableAfterDateTime
Resolves: #4759
Related: #4834 , neos/rector#42