[Ivan Pešić (eevan78)]

Summary of the Pull Request

This PR introduce the initial localization to Serbian (sr-Cyrl-RS)

References and Relevant Issues

Detailed Description of the Pull Request / Additional comments

I have added the Resources.resw files. I'm not sure if anything else should be added/changed.

Validation Steps Performed

PR Checklist

• Closes #xxx
• Tests added/passed
• Documentation updated
  • If checked, please file a pull request on our docs repo and link it here: #xxx
• Schema updated (if necessary)

[Ivan Pešić (eevan78)]

@microsoft-github-policy-service agree

[Leonard Hecker (lhecker)]

Unfortunately, we're currently integrated with Microsoft's internal localization service and that's where our translations are coming from. I suppose in theory we could merge this, but in practice it's probably not possible because of that.

[Dustin L. Howett (DHowett)]

Oh, no, as long as we are not getting Serbian translations from Touchdown we can accept community contributions! We just need a process to make sure the language stays up to date as we add new text.

[Ivan Pešić (eevan78)]

I can commit myself to update this localization. Certainly, I won't abandon it without ensuring a clear way forward with maintainers (Dustin L. Howett (@DHowett))

[Leonard Hecker (lhecker)]

After giving it some thought, we decided that we should merge your PR, because having some translation is better than having none. We'd essentially put the stewardship of keeping the translation up to date into the stewardship of the community. Our UI framework (WinUI) will fall back to English or some other secondary language if a Serbian translation string is unavailable.

[Mike Griese (zadjii-msft)]

x-linking this to #10667. This PR seems like it's the prototypical case of "if it's not one of the automatic translations, then the community is free to add their own.

Ivan Pešić (@eevan78) Is there a reason this PR is still a draft?

IIRC we have to manually list out the languages we support in either our Package.appxmanifest or... some other file. Somewhere in the settings editor, where we pick the languages? But if this works, then let's merge it

[Ivan Pešić (eevan78)]

Mike Griese (@zadjii-msft) this is my first contribution to this project, and I was not completely sure if I have picked up all places with the source strings.
Hence I marked it as a draft.

Now that there is a decision to merge it, there is no need to keep it as a draft anymore. I think that I have translated everything.

[Ivan Pešić (eevan78)]

Hello,
is there any progress on this?

[Dustin L. Howett (DHowett)]

Thank you so much! This is our first ever non-internal language contribution. We'll need to find a way to maintain these in the future, but I'm more than happy to have this one. Sorry it took so long for us to decide on this :)

[Dustin L. Howett (DHowett)]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Dustin L. Howett (DHowett)]

As a follow-up and maybe in a separate PR, you may want to play around with the language list in the Settings UI. It had to be hardcoded for annoying reasons. It's in LaunchViewModel.cpp, appLanguageTags

[Ivan Pešić (eevan78)]

Thank you so much! This is our first ever non-internal language contribution. We'll need to find a way to maintain these in the future, but I'm more than happy to have this one. Sorry it took so long for us to decide on this :)

I'm also happy to be able to contribute to this excellent project!
Don't apologize, definitely this is not a high priority change. I'm glad that it was accepted!

As a follow-up and maybe in a separate PR, you may want to play around with the language list in the Settings UI. It had to be hardcoded for annoying reasons. It's in LaunchViewModel.cpp, appLanguageTags

Sure, I will do it! This is exactly what I needed from you, a hint to find all relevant places that needs to be translated. Thanks. I'll work on this and open another PR.