Skip to content

chore(compat): [2.x] upgrade to match sylius version two#201

Merged
oallain merged 20 commits into
mainfrom
feat/sylius-v2.x
Jun 27, 2025
Merged

chore(compat): [2.x] upgrade to match sylius version two#201
oallain merged 20 commits into
mainfrom
feat/sylius-v2.x

Conversation

@maxperei

Copy link
Copy Markdown
Q A
Bug fix? no
New feature? yes
BC breaks? yes
Fixed issue #197

@maxperei maxperei requested a review from TheGrimmChester May 15, 2025 15:54
Comment thread config/config.yaml Outdated
@maxperei maxperei marked this pull request as ready for review June 12, 2025 13:58
@maxperei maxperei requested review from Jibbarth and oallain and removed request for TheGrimmChester June 12, 2025 13:58
Comment thread config/routes.yaml Outdated
Comment thread config/services.yaml Outdated
Comment thread src/Form/Type/TaxonAttributeValueType.php Outdated
Comment thread src/ProcessManager/ProcessManager.php
Comment thread tests/PHPUnit/AbstractKernelTestCase.php Outdated

@oallain oallain left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hard work, we're not far from the goal 👏🏼

Comment thread assets/controllers/form_controller.js
Comment thread config/services.yaml Outdated
Comment thread tests/PHPUnit/AbstractKernelTestCase.php Outdated

@Jibbarth Jibbarth left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Huge work! Well done 👌

@oallain oallain left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great work, well done! 👏🏼

@oallain oallain merged commit b0bdc22 into main Jun 27, 2025
13 checks passed
@oallain oallain deleted the feat/sylius-v2.x branch June 27, 2025 08:35
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.

3 participants