[maleeb]

No description provided.

[njx]

This should be removed since there isn't any such sample currently. (It will fall back to the default English sample.)

[njx]

Review complete. I can merge this when you've made these changes. Thanks.

[njx]

Oh, it looks like you also need to add an entry for this in nls/urls.js. @jasonsanjose is that right?

[jasonsanjose]

Yep. Had to do that for Finnish here #5038.

[njx]

This looks fine, but it turns out that the two links actually don't work in the About dialog because the links are actually incorrect in the strings file.

I'm going to fix that, then squash all these commits into a single update.

[njx]

Actually, it looks like we don't even have a translation for ABOUT_TEXT_WEB_PLATFORM_DOCS in the Turkish strings file, so that URL will never show up in the UI. I'll go ahead and push the other stuff.

@Cryptexx - would you consider updating the Turkish strings.js file with translations for new strings since the last time it was updated? If so, please put up a new pull request for that. Thanks.

[njx]

Merged via 4f53324; see also 63817ff. Closing.