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

Updated Italian translation #200

Open
wants to merge 1 commit into
base: 21.02
Choose a base branch
from
Open

Updated Italian translation #200

wants to merge 1 commit into from

Conversation

pynolo
Copy link

@pynolo pynolo commented Feb 18, 2022

Description

Italian translation had been left behind, many localization files expected by the updated skill were missing and they were causing Mycroft to break when a weather request was asked in Italian.
This PR adds the missing files to the it-it folder, mirroring those in the English localization.

Type of PR

  • Bugfix
  • Feature implementation
  • Refactor of code (without functional changes)
  • Documentation improvements
  • Test improvements

Testing

Documentation

No extra docs needed, this PR extends to Italian language changes already present in English.

@krisgesling
Copy link
Contributor

Thanks Pynolo - I'll merge this in however wanted to warn you that if there are any different translations in translate.mycroft.ai they will overwrite these on the next sync. It's a one-directional sync so it currently has no idea whether translations in the Skill are technically newer or not.

@krisgesling
Copy link
Contributor

Hey @pynolo I just realised that the commit is from a different github account.

It might be some kind of error as it doesn't even look like a real account - any idea what's happened here?

@pynolo
Copy link
Author

pynolo commented Mar 3, 2022 via email

@krisgesling
Copy link
Contributor

Ah cool, I'll wait until it's fixed up and then we can merge it in.

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