Add locale to the url by alecslupu · Pull Request #14432 · decidim/decidim

alecslupu · 2025-03-31T09:05:48Z

🎩 What? Why?

Note for reviewer below

In this PR we are changing the url structure of the content pages to support as the first parameter the locale.
The page like /processes/hereslug/f/57/elections/5 would become:

/en/processes/hereslug/f/57/elections/5 if seen in english
/ca/processes/hereslug/f/57/elections/5 if seen in catalan

All the main pages, would have a fallback, so, if someone has a link to /processes/hereslug/f/57/elections/5, it would be automatically redirected to either locale of the browser, user set locale, or the default language of the app

Note for the reviewer

There are many files that have been changed because i have got into a situation where, if i would have played with the default url attributes, i would have got links like : /en/processes/hereslug/f/57/elections/5?locale=en due to fact that we have multiple stacked engines, and each would need to use default, causing strange urls.

Is a big PR, but most of the changes are related to adding the locale parameter to various files.

📌 Related Issues

Link your PR to an issue

Related to #?
Fixes #?

Testing

Checkout to this branch & boot your application
Go to homepage, then start to browse the application.
You will see that all the space related pages do have a locale prefix
Check the platform pages ( help pages ) and see they also have a locale params
You can change the locale of the website, and you will see the url and locale is changing acordingly
In user profile page, all the links for resources generated by that user would have a prefix for locale

📷 Screenshots

Please add screenshots of the changes you are proposing

♥️ Thank you!

* Fix specs in initiatives * Fix specs * Fix initiatives specs

…n-param

github-actions

This pull request does not contain a valid label. Please add one of the following labels: ['type: feature', 'type: change', 'type: fix', 'type: removal', 'target: developer-experience', 'type: internal']

decidim-participatory_processes/app/cells/decidim/participatory_processes/process_g_cell.rb

.../app/views/decidim/participatory_processes/admin/participatory_process_groups/index.html.erb

decidim-proposals/spec/shared/merge_proposals_examples.rb

…y_processes/process_g_cell.rb

…y_processes/admin/participatory_process_groups/index.html.erb

andreslucena · 2025-11-24T10:10:28Z

This is ready for the review @alecslupu ?

alecslupu · 2025-11-24T10:52:25Z

This is ready for the review @alecslupu ?

@andreslucena I would say that it is ready.

…n-param

fblupi

LGTM!

andreslucena · 2026-04-09T05:34:24Z

Just skimming this a bit, but is there any reason to not doing the default_url_options approach?

Because the default_url_approach will pollute all the core urls with locale query string.

Also, we have engine, in engine in engine ( decidim_meetings is mounted in decidim_participatory_process which is mounted to the decidim_core). To have working urls, i would need to patch the each engine default_url_options.

I have tried to play with the mounted_params in both components and participatory space, and did not worked correctly.

In short, i have tried it, and did not worked as i was expecting.

Answering this at #16544 (comment) - 1 year later!! You just have to wait for the good things :D

alecslupu added 22 commits March 22, 2025 13:39

Implement url localization on the Participatory Processes

1a6b488

Implement url localization on the Assemblies

b2355a8

Implement url localization on the Conferences

465947f

Implement url localization on the Initiatives

b8a76a2

Implement url localization on the pages in core

3f90790

Fix specs

a4e7f2b

Fix specs

6f7c6e0

Fix specs

5c6e91c

Fix specs

2456868

more fixes

1e72077

Fix specs

43dc3de

Fix more specs

d0241e5

Implement url localization on the Participatory Processes (#402)

5d5cde6

* Fix specs in initiatives * Fix specs * Fix initiatives specs

Fix more specs

7efcd56

Fix specs

15d2900

Fix canonical url

7cbb2b6

Fix more specs

cc50e5e

Fix specs

3a8bdaf

Merge branch 'develop' of github.com:decidim/decidim into feature/i18…

5bf6cb4

…n-param

Merge branch 'develop' of github.com:decidim/decidim into feature/i18…

2e8236d

…n-param

Fix failing specs, improve breadcrumb

961a6d8

Running lint

7c703f7

github-actions bot requested changes Mar 31, 2025

View reviewed changes

github-actions bot added module: proposals module: core module: meetings module: budgets module: comments module: admin module: verifications labels Mar 31, 2025

alecslupu commented Nov 23, 2025

View reviewed changes

Update decidim-participatory_processes/app/cells/decidim/participator…

e32268a

…y_processes/process_g_cell.rb

alecslupu dismissed github-actions[bot]’s stale review via e32268a November 23, 2025 20:41

alecslupu added 2 commits November 23, 2025 22:41

Update decidim-participatory_processes/app/views/decidim/participator…

d489fcd

…y_processes/admin/participatory_process_groups/index.html.erb

Update decidim-proposals/spec/shared/merge_proposals_examples.rb

d66ef1f

github-actions bot previously approved these changes Nov 23, 2025

View reviewed changes

Merge branch 'develop' into feature/i18n-param

b318fe1

alecslupu dismissed github-actions[bot]’s stale review via b318fe1 November 27, 2025 07:14

github-actions bot previously approved these changes Nov 27, 2025

View reviewed changes

alecslupu mentioned this pull request Nov 28, 2025

Locale is not specified into email hyperlinks #15640

Closed

andreslucena assigned fblupi Dec 4, 2025

alecslupu added 3 commits December 4, 2025 12:22

Merge branch 'develop' of github.com:decidim/decidim into feature/i18…

bc18701

…n-param

Add RELEASE_NOTES

f9fb5c1

Fix failing specs

a6480c8

alecslupu dismissed github-actions[bot]’s stale review via a6480c8 December 4, 2025 10:42

github-actions bot previously approved these changes Dec 4, 2025

View reviewed changes

Fix failing specs

c82caa8

alecslupu dismissed github-actions[bot]’s stale review via c82caa8 December 4, 2025 11:17

github-actions bot previously approved these changes Dec 4, 2025

View reviewed changes

Fix spelling

817e58e

alecslupu dismissed github-actions[bot]’s stale review via 817e58e December 4, 2025 11:21

github-actions bot approved these changes Dec 4, 2025

View reviewed changes

fblupi approved these changes Dec 4, 2025

View reviewed changes

fblupi merged commit 9775b9f into decidim:develop Dec 4, 2025
76 checks passed

alecslupu deleted the feature/i18n-param branch December 4, 2025 18:01

andreslucena mentioned this pull request Dec 22, 2025

Fix promoting committee members page in Initiatives' admin #15844

Merged

alecslupu mentioned this pull request Apr 8, 2026

Alternative approach for fixing inconsistency issues in the locales URL #16544

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add locale to the url#14432

Add locale to the url#14432
fblupi merged 44 commits intodecidim:developfrom
i-need-another-coffee:feature/i18n-param

alecslupu commented Mar 31, 2025 •

edited

Loading

Uh oh!

github-actions bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

andreslucena commented Nov 24, 2025

Uh oh!

alecslupu commented Nov 24, 2025

Uh oh!

fblupi left a comment

Uh oh!

Uh oh!

andreslucena commented Apr 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

alecslupu commented Mar 31, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🎩 What? Why?

Note for the reviewer

📌 Related Issues

Testing

📷 Screenshots

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

andreslucena commented Nov 24, 2025

Uh oh!

alecslupu commented Nov 24, 2025

Uh oh!

fblupi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

andreslucena commented Apr 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

alecslupu commented Mar 31, 2025 •

edited

Loading