Skip to content

Support Sylius v2 - #8

Merged
oallain merged 2 commits into
masterfrom
sylius-v2
Aug 12, 2026
Merged

Support Sylius v2#8
oallain merged 2 commits into
masterfrom
sylius-v2

Conversation

@oallain

@oallain oallain commented Aug 12, 2026

Copy link
Copy Markdown
Member

No description provided.

Comment thread composer.json Outdated
Comment on lines +18 to +19
"doctrine/collections": "^2.2",
"babdev/pagerfanta-bundle": "^4.4"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can remove these deps

Comment thread composer.json Outdated
"sylius/sylius": "^1.9",
"symfony/cache": "^5.4|^6.0",
"sylius/sylius": "^2.0",
"symfony/cache": "^6.4 || ^7.3",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about authorize ^8.0 to be ready for sylius 2.3 ?

@Jibbarth Jibbarth left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ! Ship it 🚀

@oallain
oallain merged commit a0956fc into master Aug 12, 2026
1 of 12 checks passed
@oallain
oallain deleted the sylius-v2 branch August 12, 2026 12:18
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