Skip to content

[Routing] Localized Routes (i18n): Add an example for default path#20218

Closed
gbere wants to merge 1 commit intosymfony:6.4from
gbere:patch-3
Closed

[Routing] Localized Routes (i18n): Add an example for default path#20218
gbere wants to merge 1 commit intosymfony:6.4from
gbere:patch-3

Conversation

@gbere
Copy link
Copy Markdown
Contributor

@gbere gbere commented Sep 13, 2024

Localized Routes (i18n): Add an example for default path

@carsonbot carsonbot added this to the 6.4 milestone Sep 13, 2024
@gbere gbere changed the title Update routing.rst [Routing] Localized Routes (i18n): Add an example for default path Sep 13, 2024
Comment thread routing.rst
'en' => '/about-us',
'nl' => '/over-ons'
'nl' => '/over-ons',
'/about-us' // Default path for undefined locales
Copy link
Copy Markdown
Contributor

@MrYamous MrYamous Sep 13, 2024

Choose a reason for hiding this comment

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

Is that available only with attribute ? I think we should keep equivalent examples for attribute/yaml/xml/php if possible

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In YAML it doesn't work. But I added the PHP format too in #21745

@javiereguiluz
Copy link
Copy Markdown
Member

I tried to merge this but our merging tools shows multiple errors. So, I recreated this in #21745 and we're closing this PR in favor of the new one. Thanks.

javiereguiluz added a commit that referenced this pull request Jan 23, 2026
This PR was merged into the 6.4 branch.

Discussion
----------

[Routing] Default path for localized routes

I tried to merge #20218 but it failed, so let's recreate it here.

YAML format is not included because this feature doesn't work (you cannot define an entry without a key or with an empty key).

Commits
-------

cc25104 [Routing] Default path for localized routes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants