-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Update TYPO3 recipe and improve docs #4105
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
Update TYPO3 recipe and improve docs #4105
Conversation
|
Hi David, i tested the changes. Now it's working fine. But there is something else missing: The TYPO3 recipe does not include the crucial task('deploy', [
'deploy:info',
'deploy:setup',
'deploy:lock',
'deploy:release',
'typo3:update_code',
'deploy:shared',
'deploy:writable',
'deploy:vendors',
'typo3:cache:warmup',
'typo3:extension:setup',
'typo3:language:update',
'typo3:cache:flush',
// MISSING deploy:symlink
'deploy:unlock',
'deploy:cleanup',
'deploy:success',
]);The least tasks can be replaced with // recipe/common.php
task('deploy:publish', [
'deploy:symlink',
'deploy:unlock',
'deploy:cleanup',
'deploy:success',
]); |
|
Any update on this? |
MarcoRemy
left a comment
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.
In recipe/typo3.php replace lines 216-218 with deploy:publish or put deploy:symlink before
|
@antonmedv please merge this PR. I will create a fix for the missing task. |
|
Hey @MarcoRemy, thanks for jumping in, I was afk for a couple of weeks. Happy that this is complete now :) |
|
Will release a new v8 beta soon |
bin/typo3path and improved documentationThis PR updates the TYPO3 Deployer recipe to standardize CLI calls using
{{bin/typo3}}instead of hard-coded paths, ensuring compatibility with custom composer settings. Also inline PHPDoc comments and a usage example at the top of the file were included to improve maintainability and make the deployment process clearer for future developers.