Skip to content
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

Shopware recipe #3799

Open
Schrank opened this issue Apr 3, 2024 · 5 comments
Open

Shopware recipe #3799

Schrank opened this issue Apr 3, 2024 · 5 comments
Assignees

Comments

@Schrank
Copy link
Contributor

Schrank commented Apr 3, 2024

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

@Schrank Schrank self-assigned this Apr 3, 2024
@UlrichThomasGabor
Copy link
Contributor

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

@Schrank
Copy link
Contributor Author

Schrank commented Apr 3, 2024

@UlrichThomasGabor
Copy link
Contributor

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?

@antonmedv
Copy link
Member

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.

@Schrank
Copy link
Contributor Author

Schrank commented Apr 3, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants