-
Notifications
You must be signed in to change notification settings - Fork 351
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
Merged
alexander-schranz
merged 4 commits into
sulu:3.0
from
alexander-schranz:feature/route-handling-content-integration
Apr 7, 2025
Merged
Add new routing to new content integration #7749
alexander-schranz
merged 4 commits into
sulu:3.0
from
alexander-schranz:feature/route-handling-content-integration
Apr 7, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
17 tasks
e9b83e3
to
64c47e1
Compare
1f9fa61
to
832fa7f
Compare
832fa7f
to
90f27ae
Compare
90f27ae
to
519be80
Compare
This was referenced Mar 31, 2025
c64cda3
to
c1a716e
Compare
5d75e17
to
b0212f2
Compare
b0212f2
to
f555c2e
Compare
f555c2e
to
6a31b4b
Compare
d046c99
to
2267aad
Compare
Prokyonn
reviewed
Apr 7, 2025
packages/page/src/Infrastructure/Sulu/Route/WebspaceSiteRouteGenerator.php
Outdated
Show resolved
Hide resolved
packages/route/src/Infrastructure/Doctrine/Repository/RouteRepository.php
Show resolved
Hide resolved
…enerator.php Co-authored-by: Prokyonn <daniel.mathis@sector8.eu>
Prokyonn
approved these changes
Apr 7, 2025
As there exists now both routing tables for the alpha part we need change the PHPCR MIgration bundle to use |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
/test
and Site B/test
and article parent Site A/test/article
Related Pull Requests