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

Reflect Sylius Codebase Changes #1036

Merged
merged 5 commits into from
Oct 18, 2024
Merged

Conversation

Rafikooo
Copy link
Contributor

@Rafikooo Rafikooo commented Oct 16, 2024

After changes in the sylius/sylius package, some adjustments are necessary to make the standard functionality work again.

@Rafikooo Rafikooo requested review from a team as code owners October 16, 2024 14:34
@Rafikooo Rafikooo force-pushed the reflect-sylius-changes branch from 92beea2 to e9800a3 Compare October 16, 2024 22:00
@Rafikooo Rafikooo force-pushed the reflect-sylius-changes branch from 041eb2f to b2d4e3e Compare October 18, 2024 04:20
@Rafikooo Rafikooo force-pushed the reflect-sylius-changes branch from b2d4e3e to fa06445 Compare October 18, 2024 06:22
@Rafikooo Rafikooo force-pushed the reflect-sylius-changes branch from fa06445 to 2e02c46 Compare October 18, 2024 09:08
@GSadee GSadee merged commit b9de136 into Sylius:2.0 Oct 18, 2024
4 checks passed
mpysiak added a commit that referenced this pull request Oct 23, 2024
Continuation of #1036

Fixes:
1. Sylius service ID changes
<img width="1068" alt="image"
src="https://github.com/user-attachments/assets/41a3a0f7-693d-4365-a118-b532d1106869">
2. Behat `sylius.generator.image_path` service ID correct override
<img width="1008" alt="image"
src="https://github.com/user-attachments/assets/bc3576a8-0073-4c0d-b312-25b723de37fb">
3. Proper assets build
<img width="957" alt="image"
src="https://github.com/user-attachments/assets/29fdda5f-65e1-4bef-b048-59b2e58f342c">
4. Sylius registered bundles changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants