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

Redirect to page edit view when submitting for a single locale #7941

Merged
merged 1 commit into from Feb 22, 2022

Conversation

mixxorz
Copy link
Contributor

@mixxorz mixxorz commented Feb 9, 2022

When an editor decides to submit a page for translation and only chooses one locale, they are redirected to the new translated page's edit view. This behaviour is the same for snippets.

The behaviour is unchanged when translating pages/snippets to multiple locales.

This is the simple_translations counterpart to this change on wagtail-localize: wagtail/wagtail-localize#518

When an editor decides to submit a page for translation and only chooses
one locale, they are redirected to the new translated page's edit view.
@squash-labs
Copy link

squash-labs bot commented Feb 9, 2022

Manage this branch in Squash

Test this branch here: https://mixxorzpost-submit-translation-dkvnh.squash.io

Copy link
Contributor

@zerolab zerolab left a comment

Choose a reason for hiding this comment

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

Tested locally and LGTM!

@zerolab zerolab added the component:i18n i18n for content created in Wagtail, not the admin UI itself label Feb 9, 2022
@mixxorz
Copy link
Contributor Author

mixxorz commented Feb 10, 2022

One check was unsuccessful. Seems to be the one running against django@main. I assume this is sort of expected as this is running against unreleased Django code?

TODO:

  • [ ] Update changelog

@zerolab
Copy link
Contributor

zerolab commented Feb 10, 2022

@mixxorz yes, the failing tests against Django main come from taggit as far as I can tell. This is an expected failure, it is just that GitHub Actions has no good way of indicating that yet

We update the changelog after merge, so nothing to worry about there

@lb- lb- merged commit 11ecb3b into wagtail:main Feb 22, 2022
lb- added a commit that referenced this pull request Feb 22, 2022
@lb-
Copy link
Member

lb- commented Feb 22, 2022

@mixxorz great work, thanks for putting this together and also for the robust tests

@mixxorz mixxorz deleted the post-submit-translation-redirect branch February 22, 2022 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:i18n i18n for content created in Wagtail, not the admin UI itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants