Siri intent definition localisation #214
Conversation
| "Check the weather", | ||
| comment: "Siri suggestion phrase to check weather") | ||
| private static let checkWeatherSuggestedPhrase = NSString | ||
| .deferredLocalizedIntentsString(with: "CheckWeatherSuggestedPhrase") as String |
There was a problem hiding this comment.
This new concept of deferred localisation enables a donated intent to be localised potentially much later, when the device could have changed language.
| @@ -39,6 +39,10 @@ | |||
| </dict> | |||
| <key>NSPhotoLibraryUsageDescription</key> | |||
| <string>Tropos does not access your photo library.</string> | |||
| <key>NSUserActivityTypes</key> | |||
| <array> | |||
| <string>CheckWeatherIntent</string> | |||
There was a problem hiding this comment.
This was automatically added by Xcode.
| @@ -0,0 +1,6 @@ | |||
| "8SKsHj" = "Check weather forecast"; | |||
There was a problem hiding this comment.
Needs translation into Italian.
|
|
||
| "Hpkcrr" = "${conditionsDescription}"; | ||
|
|
||
| "RwwC7B" = "Check the current weather forecast"; |
There was a problem hiding this comment.
Needs translation into Italian.
| @@ -0,0 +1,6 @@ | |||
| "8SKsHj" = "Check weather forecast"; | |||
There was a problem hiding this comment.
Needs translation into Polish.
|
|
||
| "Hpkcrr" = "${conditionsDescription}"; | ||
|
|
||
| "RwwC7B" = "Check the current weather forecast"; |
There was a problem hiding this comment.
Needs translation into Polish.
| @@ -0,0 +1,6 @@ | |||
| "8SKsHj" = "Check weather forecast"; | |||
There was a problem hiding this comment.
Needs translation into Swedish.
|
|
||
| "Hpkcrr" = "${conditionsDescription}"; | ||
|
|
||
| "RwwC7B" = "Check the current weather forecast"; |
There was a problem hiding this comment.
Needs translation into Swedish.
|
Hello former Tropos contributors! Would you be interested in contributing translations for Tropos’ new custom Siri shortcut?
If you're able to help out, thanks! The new strings are located at |
|
For anyone contributing translations, could you also take a look at |
|
This is probably terrible, but for now I've added Polish and Swedish via Google Translate, in order to try and get something shipped for the iOS 12 launch. I'll be travelling for a few weeks, but I'll try and get these translations checked/improved once I return. |
sharplet
changed the title
I've checked and the Polish translations look good. 👍 |
|
@sharnik Thank you! |
sharplet
force-pushed
the
intent-localisation
branch
4 times, most recently
from
fa72d91
to
bdd4ff9
Compare
|
There's something up with the simulators on Circle CI right now. The tests are passing on my machine, but I don't currently have the GM installed, so I'll have to wait before I can verify using the GM (unless someone else wants to have a go). |
sharplet
force-pushed
the
intent-localisation
branch
from
bdd4ff9
to
044d770
Compare
sharplet
force-pushed
the
intent-localisation
branch
from
044d770
to
c8231cc
Compare
When I submitted an initial beta build to TestFlight, I got a warning about missing localisations for our supported languages (Italian, French and Polish). This work-in-progress PR first configures the project for Base Internationalisation, then sets up strings files for our various languages. This branch can then become a staging area for the actual translations, which I'll be reaching out to previous contributors for help with.