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.

Sign up for GitHub

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

Jump to bottom

Reflect Sylius 2.0 Codebase Changes - Continuation #1037

Merged
merged 8 commits into from
Oct 23, 2024
Merged

Reflect Sylius 2.0 Codebase Changes - Continuation #1037

merged 8 commits into from
Oct 23, 2024

Conversation

Rafikooo
Copy link
Contributor

@Rafikooo Rafikooo commented Oct 18, 2024 •
edited
Loading

Continuation of #1036

Fixes:

  1. Sylius service ID changes
image 2. Behat `sylius.generator.image_path` service ID correct override image 3. Proper assets build image 4. Sylius registered bundles changes

@Rafikooo Rafikooo requested review from a team as code owners October 18, 2024 13:19
@Rafikooo Rafikooo changed the title Fix Default Locale Route Fix Default Locale Route & Remove SonataBlockBundle From Registered Bundles Oct 22, 2024
@Rafikooo Rafikooo changed the title Fix Default Locale Route & Remove SonataBlockBundle From Registered Bundles Fix Default Locale Route & Remove SonataBlockBundle From Registered Bundles Oct 22, 2024
@Rafikooo Rafikooo changed the title Fix Default Locale Route & Remove SonataBlockBundle From Registered Bundles Reflect Sylius 2.0 Codebase Changes Oct 23, 2024
@Rafikooo Rafikooo changed the title Reflect Sylius 2.0 Codebase Changes Reflect Sylius 2.0 Codebase Changes - Continuation Oct 23, 2024
@mpysiak mpysiak merged commit ac33643 into Sylius:2.0 Oct 23, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wojdylak Wojdylak Wojdylak approved these changes

@NoResponseMate NoResponseMate NoResponseMate approved these changes

@lchrusciel lchrusciel lchrusciel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Rafikooo @lchrusciel @NoResponseMate @Wojdylak @mpysiak