-
Notifications
You must be signed in to change notification settings - Fork 66
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 StaticContentModule #4369
Add StaticContentModule #4369
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
Any idea where to provide the permission scheme (component ZikulaCoreBundle::
, instance static::
)? Maybe in the ExtensionsModule?
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
agreed; it could also be populated programmatically ( |
This comment has been minimized.
This comment has been minimized.
I think this would be a use case for adding a custom route (Routes module) to let |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
604ac40
to
0c562ec
Compare
@Guite I am having second thoughts about removing the contents blocks from Blocks module at 3.1.0. I am thinking full removal should wait until 4.0. In the meantime, I think we should add StaticContent module and deprecate the old blocks with plans to remove at 4.0. In the meantime, we can post an alert to the blocks admin that encourages user to move blocks to StaticContent. Then we don't do anything in the upgrade, etc. thoughts? |
That would indeed be cleaner. Good idea. |
modify docs remove all content-related blocks (move to new StaticContentModule)
terrific 😃 |
Description
Added template display controller for quick generation and display of static template-based content.To do
- [x] docs- [x] also add template-based block- [x] address question of required/p/*
in route- [ ] add permission scheme somewheremake:zikula-extension
) to discourage use of Zikula as vendor- [ ] update BlocksModuleInstaller to shift new block creation on install to StaticContentModule and update existing classes.