Using new routing system from Sylius resource bundle by loic425 · Pull Request #18213 · Sylius/Sylius

loic425 · 2025-07-07T13:23:46Z

Q	A
Branch?	2.1
Bug fix?	no
New feature?	yes
BC breaks?	no
Deprecations?	no
Related tickets	fixes #18212
License	MIT

coderabbitai · 2025-07-07T13:23:53Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch new-routing

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

github-actions · 2025-07-17T06:53:54Z

❗ Preview Environment deployment failed on Bunnyshell

See: Environment Details | Pipeline Logs

Check https://github.com/Sylius/Sylius/actions/runs/22478151407 for details.

Available commands:

🚀 /bns:deploy to redeploy the environment
❌ /bns:delete to remove the environment

Removes old upmerge paths (`1.14 -> 2.1`, `2.1 -> 2.2`) and sets up upmerge from `2.2` to `2.3`. Also updates the workflow file link in PR body to point to the `2.3` branch.  ## Summary by CodeRabbit * **Chores** * Updated the upmerge workflow configuration to refine version merge scope and refresh reference links. ✏️ Tip: You can customize this high-level summary in your review settings.

This PR has been generated automatically. For more details see [upmerge_pr.yaml](/Sylius/Sylius/blob/2.3/.github/workflows/upmerge_pr.yaml). **Remember!** The upmerge should always be merged with using `Merge pull request` button. In case of conflicts, please resolve them manually with usign the following commands: ``` git fetch upstream gh pr checkout <this-pr-number> git merge upstream/2.3 -m "Resolve conflicts between 2.2 and 2.3" ``` If you use other name for the upstream remote, please replace `upstream` with the name of your remote pointing to the `Sylius/Sylius` repository. Once the conflicts are resolved, please run `git merge --continue` and push the changes to this PR.  ## Summary by CodeRabbit * **Chores** * Updated PSR-7 HTTP message dependency constraint to support both version 1.x and 2.x. * Corrected error page template file path case sensitivity references. ✏️ Tip: You can customize this high-level summary in your review settings.

This PR has been generated automatically. For more details see [upmerge_pr.yaml](/Sylius/Sylius/blob/2.3/.github/workflows/upmerge_pr.yaml). **Remember!** The upmerge should always be merged with using `Merge pull request` button. In case of conflicts, please resolve them manually with usign the following commands: ``` git fetch upstream gh pr checkout <this-pr-number> git merge upstream/2.3 -m "Resolve conflicts between 2.2 and 2.3" ``` If you use other name for the upstream remote, please replace `upstream` with the name of your remote pointing to the `Sylius/Sylius` repository. Once the conflicts are resolved, please run `git merge --continue` and push the changes to this PR.  ## Summary by CodeRabbit * **New Features** * Consistent form field help display across widgets; improved admin taxon parent autocomplete (case-insensitive). * **Bug Fixes** * Fixed a template variable typo preventing form rendering. * Avoided incorrect category display when a main taxon name is missing. * **Localization** * Added/updated German translations for administration password reset and Romanian pluralization tweaks. * **Documentation** * Expanded contributor guidelines with detailed best practices and patterns. * **Tests** * Added unit and UI tests covering localhost resolution and taxonomy behaviors. ✏️ Tip: You can customize this high-level summary in your review settings.

* 2.2: Change application's version to v1.14.17-dev Generate changelog for v1.14.16 Change application's version to v1.14.16

* 2.2: Change application's version to v2.1.12-dev Generate changelog for v2.1.11 Change application's version to v2.1.11

Increases `timeout-minutes` from 15 to 20 minutes across all CI workflow files to reduce job cancellations. Ref: #18693  ## Summary by CodeRabbit * **Chores** * Extended timeout limits for continuous integration and deployment jobs from 15 to 20 minutes across multiple workflow configurations to allow additional execution time. ✏️ Tip: You can customize this high-level summary in your review settings.

This PR has been generated automatically. For more details see [upmerge_pr.yaml](/Sylius/Sylius/blob/2.3/.github/workflows/upmerge_pr.yaml). **Remember!** The upmerge should always be merged with using `Merge pull request` button. In case of conflicts, please resolve them manually with usign the following commands: ``` git fetch upstream gh pr checkout <this-pr-number> git merge upstream/2.3 -m "Resolve conflicts between 2.2 and 2.3" ``` If you use other name for the upstream remote, please replace `upstream` with the name of your remote pointing to the `Sylius/Sylius` repository. Once the conflicts are resolved, please run `git merge --continue` and push the changes to this PR.  ## Summary by CodeRabbit * **Chores** * Bumped release version and updated internal version identifiers for the next patch. * Added a new changelog entry for v2.2.2 (2026-01-20). * **Tests** * Updated frontend test metadata to run with the Mink Chromedriver setup. ✏️ Tip: You can customize this high-level summary in your review settings.

| Q | A |-----------------|----- | Branch? | 2.3 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | - | License | MIT  ## Summary by CodeRabbit * **Chores** * Added GitHub Actions workflow automation for package splitting and distribution. * Introduced build script to generate package matrix configuration for automated package management. ✏️ Tip: You can customize this high-level summary in your review settings.

Use new unified database format in e2e workflows: - database: "mysql:X.X" instead of database+database_version - database: "postgres:X.X" instead of postgresql+database_version - database: "mariadb:X.X" instead of mariadb+database_version - Remove DATABASE_URL from env (action sets it automatically)

Testing new BuildTestAppAction from `feature/auto-database-url` branch which introduces: - Unified database format: `mysql:8.4` instead of separate `database` + `database_version` - Automatic `DATABASE_URL` setting (no need to define in workflows) - Unified credentials (`sylius:sylius@sylius`) across all databases Changes in this PR: - Update ci_e2e-mysql.yaml, ci_e2e-pgsql.yaml, ci_e2e-mariadb.yaml - Remove `DATABASE_URL` from env sections - Use new `database: "type:version"` format Related: SyliusLabs/BuildTestAppAction#19  ## Summary by CodeRabbit * **Chores** * CI workflows updated to use automated database configuration for MariaDB, MySQL, and PostgreSQL. * Removed hard-coded database connection environment entries so jobs rely on generated configuration. * Standardized database selection input across matrix jobs and updated build steps to a newer build-action for more consistent test runs. ✏️ Tip: You can customize this high-level summary in your review settings.

…index route

| Q | A |-----------------|----- | Branch? | new-routing | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Related tickets | partially #18212 | License | MIT

Co-authored-by: Jan Góralski <jan.wojciech.goralski@gmail.com>

| Q | A |-----------------|----- | Branch? | new-routing | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Related tickets | partially #18212 | License | MIT  Inspired by the [ResetPasswordAction](https://github.com/Sylius/Sylius/blob/0f2e2b33dc222b27d911e73a41afc226027b877d/src/Sylius/Bundle/AdminBundle/Action/Account/ResetPasswordAction.php) from the admin panel.

| Q | A |-----------------|----- | Branch? | new-routing | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Related tickets | partially #18212 | License | MIT

loic425 requested review from a team as code owners July 7, 2025 13:23

loic425 mentioned this pull request Jul 7, 2025

Using new routing system from Sylius resource bundle #18212

Open

loic425 marked this pull request as draft July 7, 2025 13:25

loic425 changed the title ~~Bump Sylius resource versions~~ Using new routing system from Sylius resource bundle Jul 7, 2025

loic425 force-pushed the new-routing branch from b41e54a to 6bc40da Compare July 7, 2025 13:28

loic425 force-pushed the new-routing branch from 6bc40da to b4580df Compare July 16, 2025 18:07

loic425 force-pushed the new-routing branch from b58ab5c to 7ba1dbc Compare August 27, 2025 09:03

loic425 force-pushed the new-routing branch from e65c9e8 to 879cb26 Compare November 5, 2025 14:06

probot-autolabeler bot added the Maintenance CI configurations, READMEs, releases, etc. label Nov 5, 2025

loic425 changed the base branch from 2.1 to 2.2 November 5, 2025 14:11

probot-autolabeler bot added the Admin AdminBundle related issues and PRs. label Nov 27, 2025

loic425 force-pushed the new-routing branch 2 times, most recently from fde6308 to 179d7ea Compare January 14, 2026 07:35

Rafikooo and others added 7 commits January 16, 2026 12:47

[CI] Update upmerge workflow to 2.2 -> 2.3

c45bf26

[CI] Increase job timeouts to 20 minutes

be8d656

Merge branch '2.2' into 2.3

ad20a38

* 2.2: Change application's version to v1.14.17-dev Generate changelog for v1.14.16 Change application's version to v1.14.16

Merge branch '2.2' into 2.3

198943e

* 2.2: Change application's version to v2.1.12-dev Generate changelog for v2.1.11 Change application's version to v2.1.11

loic425 force-pushed the new-routing branch from 605c50c to 19269e4 Compare January 21, 2026 15:26

TheMilek and others added 6 commits January 22, 2026 15:03

[Maintenance][CI] Add monorepo split action

20e3c06

loic425 added 20 commits February 26, 2026 09:15

[New Routing] Fix notification message for order shipment

ff8ac1b

[New routing] Delete multiple products from a taxon

5ba2ae5

[New Routing] Migrate channel pricing log entry routing

0b51ff9

[New routing] Convert sylius_admin_customer_order_index route

105cd6f

[New routing] Convert sylius_admin_catalog_promotion_product_variant_…

6083b2d

…index route

[New routing] Shop product review crud

72c7f79

Fix composer require checker

49fed0e

[New routing] Bump resource & package

f7e2f42

[New routing] Shop account order crud

302ed2a

[New routing] Shop product

7f926d9

[New routing] Shop account address book

24cc2e3

[New routing] Shop checkout

2adb1d7

[New routing] Shop account

54b0777

[New routing] Adjust bulk delete route paths

4cb44ef

[New routing] Shop register

4ed3ee3

Refactoring order thank you controller

30fdd20

Apply suggestions from code review

5b540b2

[New routing] Uniformize shop resources

241e472

Refactoring cart summary controller

5b83b95

[New routing] Refactoring cart checkout

9ca8f17

loic425 force-pushed the new-routing branch from ba25f53 to 9ca8f17 Compare February 26, 2026 08:15

loic425 and others added 9 commits February 26, 2026 09:16

Refactoring move product taxon positions controller

9344b15

[New routing] Reset password

8ed430a

Apply suggestions from code review

eb9a4e4

Co-authored-by: Jan Góralski <jan.wojciech.goralski@gmail.com>

[New routing] Refactoring account change password

73927f2

[New routing] Request reset password token

703b948

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Using new routing system from Sylius resource bundle#18213

Using new routing system from Sylius resource bundle#18213
loic425 wants to merge 245 commits into2.2from
new-routing

loic425 commented Jul 7, 2025

Uh oh!

coderabbitai bot commented Jul 7, 2025 •

edited

Loading

Review skipped

Uh oh!

github-actions bot commented Jul 17, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Uh oh!

Conversation

loic425 commented Jul 7, 2025

Uh oh!

coderabbitai bot commented Jul 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

github-actions bot commented Jul 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❗ Preview Environment deployment failed on Bunnyshell

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

coderabbitai bot commented Jul 7, 2025 •

edited

Loading

github-actions bot commented Jul 17, 2025 •

edited

Loading