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

Add support for enqueue 0.10 #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bfontaine
Copy link

Per php-enqueue/enqueue-dev#1321 (comment) (cc @makasim)

I’m not sure how to test this. The contribution guide in enqueue-dev mention commands that don’t exist in this repo (and it’s not listed as a split repo).

@bfontaine
Copy link
Author

I just saw this duplicates #7

@makasim
Copy link
Member

makasim commented Nov 13, 2023

can you drop all versions except for 0.10 please? They are not supported.

@makasim
Copy link
Member

makasim commented Nov 13, 2023

I’m not sure how to test this.

Let's update and see what happens

@bfontaine
Copy link
Author

ok, done.

@bfontaine
Copy link
Author

Well as-is it doesn’t work because of php-enqueue/enqueue-dev#1136. This code was removed in php-enqueue/enqueue-dev@ed707fe5 for 0.9.0. I’m not sure what to use now that it’s gone; I don’t see it in UPGRADE.md.

@makasim
Copy link
Member

makasim commented Nov 13, 2023

Sorry, I don't remember what it was about.

@bfontaine bfontaine changed the title Add support for enqueue 0.9 and 0.10 Add support for enqueue 0.10 Nov 13, 2023
@bfontaine
Copy link
Author

The Magento module depends on Enqueue\Symfony\Client\Meta\* classes, which were removed in 0.9.0.

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

Successfully merging this pull request may close these issues.

2 participants