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

Localize document type scaffolds for nested content #3696

Merged

Conversation

kjac
Copy link
Contributor

@kjac kjac commented Nov 16, 2018

Prerequisites

  • I have added steps to test this contribution in the description below

If there's an existing issue for this PR then this fixes: #3483

Description

This PR adds localization to the nested content item picker, as outlined in #3483.

Before

nc-translation-before

After

nc-translation-after

Positive, unintended side effect

As-is the document type name isn't localized on the info tab of content items:

nc-translation-doctype-info-before

With this PR, they are:

nc-translation-doctype-info-after

Copy link
Contributor

@dawoe dawoe left a comment

Choose a reason for hiding this comment

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

Hi @kjac

Finally found time to test this on. Works like a charm

Thanks again for creating this fix.

Dave Community Pull Request team

@kjac
Copy link
Contributor Author

kjac commented Dec 6, 2018

😁 👍

@nul800sebastiaan nul800sebastiaan merged commit 7149ebc into umbraco:dev-v7 Dec 9, 2018
@nul800sebastiaan
Copy link
Member

Yep, works great! 👍

@kjac kjac deleted the fix-nested-content-doctype-translation branch December 9, 2018 18:34
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 this pull request may close these issues.

3 participants