Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 97c9426

Browse files
minor #15706 Try SyliusResourceBundle 1.11 & SyliusGridBundle 1.13 (loic425)
This PR was merged into the 1.13 branch. Discussion ---------- | Q | A | |-----------------|--------------------------------------------------------------| | Branch? | 1.13 | | Bug fix? | no | | New feature? | no | | BC breaks? | no | | Deprecations? | no | | Related tickets | | | License | MIT | <!-- - Bug fixes must be submitted against the 1.12 branch - Features and deprecations must be submitted against the 1.13 branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> Inspired by #15119 Commits ------- Try SyliusGridBundle 1.13 Both resource and grid unstable versions Apply suggestions from code review [Resource] Add validation for translatable interface with new namespace Minor CS fix in workflow
2 parents 4984605 + 5f56fef commit 97c9426

2 files changed

Lines changed: 18 additions & 2 deletions

File tree

.github/workflows/ci_e2e-custom.yaml

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@ permissions:
1414
contents: read
1515

1616
jobs:
17-
behat-no-js-unstable-symfony:
17+
behat-no-js-unstable-dependencies:
1818
runs-on: ubuntu-latest
19-
name: "Non-JS, PHP ${{ matrix.php }}, Symfony ${{ matrix.symfony }}, MySQL ${{ matrix.mysql }} (Unstable Dependencies)"
19+
name: "Non-JS, PHP ${{ matrix.php }}, Symfony ${{ matrix.symfony }}, MySQL ${{ matrix.mysql }}, Resource ${{ matrix.resource }}, Grid ${{ matrix.grid }} (Unstable Dependencies)"
2020
timeout-minutes: 45
2121
continue-on-error: true
2222
strategy:
@@ -25,6 +25,8 @@ jobs:
2525
php: [ "8.1" ]
2626
symfony: [ "^6.4" ]
2727
mysql: [ "8.0" ]
28+
resource: ["^1.11@dev"]
29+
grid: [ "^1.13@dev" ]
2830

2931
env:
3032
APP_ENV: test_cached
@@ -57,6 +59,14 @@ jobs:
5759
composer config minimum-stability dev
5860
composer config prefer-stable true
5961
62+
- name: Require SyliusResourceBundle version
63+
run: |
64+
composer require sylius/resource-bundle:${{ matrix.resource }} --no-update
65+
66+
- name: Require SyliusGridBundle version
67+
run: |
68+
composer require sylius/grid-bundle:${{ matrix.grid }} --no-update
69+
6070
- name: Build application
6171
uses: SyliusLabs/[email protected]
6272
with:

src/Sylius/Bundle/CoreBundle/Resources/config/validation/Translatable.xml renamed to src/Sylius/Bundle/CoreBundle/Resources/config/validation/TranslatableInterface.xml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,10 @@
1717
<option name="groups">sylius</option>
1818
</constraint>
1919
</class>
20+
21+
<class name="Sylius\Resource\Model\TranslatableInterface">
22+
<constraint name="Sylius\Bundle\CoreBundle\Validator\Constraints\TranslationForExistingLocales">
23+
<option name="groups">sylius</option>
24+
</constraint>
25+
</class>
2026
</constraint-mapping>

0 commit comments

Comments
 (0)