shopware / production

Shopware 6 production template
https://shopware.com
177 stars 151 forks source link

Better extendability for sytem:setup command and extend the one from MaintenanceBundle #122

Closed AndreasA closed 3 years ago

AndreasA commented 3 years ago

Please describe the feature you would like to see implemented.

Instead of copying the command from the MaintenaceBundle it would be better if it would just extend the one from the MaintenanceBundle and add some extendability options like some protected methods that could be overridden, e.g. to add custom env variables etc. just before confirmation.

it would also make it easier to update those commands.

mitelg commented 3 years ago

we are completely moving such commands and scripts to the platform repository. the ones in the production template are still there, due to backward compatibility reasons. and will be removed in the future. correct me if I'm wrong @keulinho

AndreasA commented 3 years ago

@mitelg It makes sense to move them there completely.

However, maybe you could add two events to the setup command:

That way the command is easily customizable for a project.

Regarding backward compatibility, just wondering but the commands have to be manually updated by copying the new files to an existing project, so wouldn't removing them and adding the Maintenance bundle be easier for a user during the update/upgrade process?

shyim commented 3 years ago

https://github.com/shopware/production/commit/4a0cd7c022c3971d7e8dacaa4a1ec5fa6c954f2e

We had it in pipeline already for months is now merged :sweat_smile: