deployer icon indicating copy to clipboard operation
deployer copied to clipboard

Shopware recipe

Open Schrank opened this issue 1 year ago • 5 comments

I'm currently trying the current shopware recipe and I think I'm missing something.

Three things I don't understand and would love to fix (therefore an issue an no discussion):

  • The JS is build locally but not uploaded - isn't it?
  • composer install is not called and uploaded

Am I missing something or is the recipe incomplete?

My current fix is to call composer install on the remote host and I ran theme:compile manually. If you explain to me how it is expected to work I'm happy to fix it - either the docs or the code :-)

cc @UlrichThomasGabor @peterjaap

Upvote & Fund

  • We're using Polar.sh so you can upvote and help fund this issue.
  • We receive the funding once the issue is completed & confirmed by you.
  • Thank you in advance for helping prioritize & fund our backlog.
Fund with Polar

Schrank avatar Apr 03 '24 08:04 Schrank

My intention was to not build anything on the remote host. composer install is missing and could be added, but I'd call it locally for my use-case.

JS should be uploaded? Whole directory should be uploaded? deploy:update_code should do that

UlrichThomasGabor avatar Apr 03 '24 09:04 UlrichThomasGabor

Thanks for the quick answer! https://github.com/deployphp/deployer/blob/master/recipe/deploy/update_code.php doesn't contain any upload, maybe you are using https://github.com/deployphp/deployer/blob/master/contrib/rsync.php ?

Schrank avatar Apr 03 '24 09:04 Schrank

Ah, hm.

Yes, I have overwritten 'deploy:update_code' to use upload(). But this function seems to be not really in wide use.

Question is, what is the goal of this recipe.

I have mainly used it to build locally and push that onto a remote server. If this is the goal, then I guess update_code should be replaced with rsync or something like that, such that the recipe works without overwriting tasks.

Building on the remote server, I'd actually try to achieve with shopware build-ins? Especially with the growing support of composer, it seems to me that the board means are there to achieve that and a deployer recipe is not necessary?

UlrichThomasGabor avatar Apr 03 '24 10:04 UlrichThomasGabor

I have mainly used it to build locally and push that onto a remote server. If this is the goal, then I guess update_code should be replaced with rsync or something like that, such that the recipe works without overwriting tasks.

Yeap. Lets update the recipe.

antonmedv avatar Apr 03 '24 18:04 antonmedv

Will take care of it. Have a working prototype, except the theme is not used on production 🤪

Schrank avatar Apr 03 '24 18:04 Schrank

This issue has been automatically closed. Please, open a discussion for bug reports and feature requests.

Read more: [https://github.com/deployphp/deployer/discussions/3888]

github-actions[bot] avatar Sep 09 '24 20:09 github-actions[bot]

This issue has been automatically closed. Please, open a discussion for bug reports and feature requests.

Read more: [https://github.com/deployphp/deployer/discussions/3888]

github-actions[bot] avatar Sep 09 '24 20:09 github-actions[bot]