Skip to content
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

Do not add Aliases for translated pages if they are not ticked for actual translation #15063

Closed
2 tasks
stevejalim opened this issue Sep 2, 2024 · 2 comments
Closed
2 tasks

Comments

@stevejalim
Copy link
Collaborator

stevejalim commented Sep 2, 2024

By default when a user adds a new page in en-US, wagtail-localize adds an Alias page in each of the (10) configured languages that we currently support. However, the problem is that if you go to /fr/some-new-page/, the URL path stays the same, but displays the unlocalized content from /en-US/some-new-page.

This automatic creation of Aliases is done in order to preserve a synchronised page tree between locales, but it doesn't really suit how we want our page tree to be, and the Aliasing behaviour is not what we want - if there is no page specifically published for /fr/some-new-page/ we should 404, rather than show the content of the en-US version of that page.

As such, we need to find a way to disable the auto-generation of aliases in a clean, clear and documented way.


Examples of other people with this issue:


Success criteria

  • Page aliases are not automatically created for new pages
  • Page aliases are not creatable at all via the UI
@stevejalim
Copy link
Collaborator Author

Note that completely disabling the Alias would also be a (somewhat sledgehammer) solution to mozilla/wagtail-localize-smartling#22, too

@stevejalim
Copy link
Collaborator Author

Well, that was a useful Zoom with @alexgibson - while helping him set up a local state to test #15066 we spotted that we could just not enable locale syncronisation for a Locale... and all the pain goes away.

Closing as invalid / StevePEBCAK

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 a pull request may close this issue.

1 participant