WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@mamazu
Copy link
Contributor

@mamazu mamazu commented Sep 8, 2025

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

What's in this PR?

Replacing the rest routing with Symfony routing.

Why?

Otherwise it will break in 3.0

@alexander-schranz alexander-schranz changed the title Migreate rest routing to symfony routing Migrate rest routing to symfony routing Sep 8, 2025
* Controller to create dynamic form entries list.
*/
class DynamicController implements ClassResourceInterface
class DynamicController
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we removed in any bundles yet the ClassResourceInterface on 2.x so we should keep it here.

@mamazu mamazu force-pushed the replacing_routing branch 2 times, most recently from a55e4df to 66cc21f Compare November 7, 2025 14:00
@mamazu mamazu force-pushed the replacing_routing branch from 66cc21f to 3accba8 Compare November 7, 2025 14:00
Updated version number and deprecated fos rest routing.
@alexander-schranz alexander-schranz merged commit 8daeb31 into sulu:2.5 Nov 10, 2025
8 checks passed
@mamazu mamazu deleted the replacing_routing branch November 10, 2025 13:52
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