Skip to content

Add new routing to new content integration #7749

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

alexander-schranz
Copy link
Member

@alexander-schranz alexander-schranz commented Jan 24, 2025

Q A
Bug fix? no
New feature? yes
BC breaks? yes
Deprecations? yes
Fixed tickets fixes #
Related issues/PRs #7726, sulu/skeleton#276
License MIT
Documentation PR sulu/sulu-docs#

What's in this PR?

Add new route implementation for the content bundle. This PR adds a few hacks and things we should revert before the stable release. Most things where added to keep compatibility to older bundles and routings.

Why?

Created the new route implementation in #7726

ToDo: Done in #7726

Related Pull Requests

@alexander-schranz alexander-schranz changed the base branch from 2.6 to 3.0 January 24, 2025 12:55
@alexander-schranz alexander-schranz added the Feature New functionality not yet included in Sulu label Jan 24, 2025
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch from e9b83e3 to 64c47e1 Compare February 28, 2025 08:02
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch 9 times, most recently from 1f9fa61 to 832fa7f Compare March 26, 2025 10:18
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch from 832fa7f to 90f27ae Compare March 31, 2025 11:49
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch from 90f27ae to 519be80 Compare March 31, 2025 13:40
@alexander-schranz alexander-schranz changed the title Feature/route handling content integration Add new routing to new content integration Apr 1, 2025
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch 5 times, most recently from c64cda3 to c1a716e Compare April 1, 2025 13:45
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch 2 times, most recently from 5d75e17 to b0212f2 Compare April 1, 2025 15:19
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch from b0212f2 to f555c2e Compare April 1, 2025 15:39
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch from f555c2e to 6a31b4b Compare April 1, 2025 15:46
@alexander-schranz alexander-schranz force-pushed the feature/route-handling-content-integration branch from d046c99 to 2267aad Compare April 1, 2025 15:53
@alexander-schranz alexander-schranz marked this pull request as ready for review April 7, 2025 10:09
…enerator.php

Co-authored-by: Prokyonn <daniel.mathis@sector8.eu>
@alexander-schranz alexander-schranz merged commit a515b9e into sulu:3.0 Apr 7, 2025
9 checks passed
@alexander-schranz alexander-schranz deleted the feature/route-handling-content-integration branch April 7, 2025 14:51
@alexander-schranz
Copy link
Member Author

alexander-schranz commented Apr 7, 2025

As there exists now both routing tables for the alpha part we need change the PHPCR MIgration bundle to use ro_routes_next table instead of ro_routes and migrate it to that. @Prokyonn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New functionality not yet included in Sulu
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants