Skip to content

Deprecate rest routing and provide Symfony Routing alternative#79

Open
mamazu wants to merge 3 commits intosulu:2.2from
mamazu:replacing_routing
Open

Deprecate rest routing and provide Symfony Routing alternative#79
mamazu wants to merge 3 commits intosulu:2.2from
mamazu:replacing_routing

Conversation

@mamazu
Copy link
Contributor

@mamazu mamazu commented Aug 19, 2025

Q A
Bug fix? no
New feature? no
BC breaks? yes
Deprecations? no
Fixed tickets -
Related issues/PRs refs sulu/sulu#7434
License MIT

What's in this PR?

Replacing rest routing with Symfony routing.

Why?

It's not part of sulu anymore and will be removed in 3.0 anyways.

@mamazu mamazu force-pushed the replacing_routing branch 3 times, most recently from 24bc36f to 39025a6 Compare August 19, 2025 15:23
@mamazu mamazu force-pushed the replacing_routing branch from 39025a6 to b28834b Compare August 19, 2025 15:26
@mamazu mamazu force-pushed the replacing_routing branch 4 times, most recently from 9e35f0a to d473025 Compare August 19, 2025 15:50
@mamazu
Copy link
Contributor Author

mamazu commented Aug 19, 2025

This has the same issue with formatting than the one in the community bundle. If I would fix the pull request the 7.x build breaks.

@mamazu mamazu force-pushed the replacing_routing branch from d473025 to 00623f3 Compare August 19, 2025 15:59
@mamazu mamazu changed the title Replacing routing Replacing fos rest routing with Symfony Routing Aug 19, 2025
@mamazu mamazu force-pushed the replacing_routing branch 2 times, most recently from e1f91dd to 5027a5e Compare August 19, 2025 16:05
@mamazu mamazu force-pushed the replacing_routing branch 2 times, most recently from bfeb1c7 to db34c8f Compare December 16, 2025 20:35
@mamazu mamazu changed the title Replacing fos rest routing with Symfony Routing Deprecate rest routing and provide Symfony Routing alternative Dec 16, 2025
@alexander-schranz
Copy link
Member

alexander-schranz commented Dec 17, 2025

Fine to set the PHP min requirement to 8.2 and Sulu 2.6 if but atleast one job should run on lowest.

@alexander-schranz
Copy link
Member

disable trailing_comma_in_multiline CS Rule unnecessary change for this PR.

@mamazu mamazu changed the base branch from 2.2 to 3.0 February 10, 2026 16:47
@mamazu mamazu changed the base branch from 3.0 to 2.2 February 10, 2026 16:48
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