-
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
Include default typoscript installation-wide #55
Conversation
Instead of this workaround, you should probably include the TypoScript of this extension in your Sites TypoScript Add |
@Kanti, that would work for Frontend, but not for Backend, wouldn't it? |
072dddf
to
04df60c
Compare
04df60c
to
b778caa
Compare
I've changed the code, removed the initializeView part, and added an ext_localconf.php file which includes the typoscript installation-wide. I had some warnings when opening up the backend module. This fixes it. |
My workaround was also to add the proper TypoScript in our SitePackage (TYPO3 11.5) Once this get's merged I can remove my workaround in the project. Just for the reference, I had to provide key, type and content: \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript(
'mail_logger',
'setup',
'@import \'EXT:mail_logger/Configuration/TypoScript/setup.typoscript\''
);
\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript(
'mail_logger',
'constants',
'@import \'EXT:mail_logger/Configuration/TypoScript/constants.typoscript\''
); |
I had time to test it with our Projects, did does not break anything, so I will release this. |
@Kanti, thx |
Currently the dashboard template cannot be loaded, see #35. This fixes it.
Temporary workaround: patch via composer-patches:
Fixes #35