Make sure languages "with two parts" are handled correctly #17

Open
opened 5 years ago by elegaanz · 3 comments
elegaanz commented 5 years ago (Migrated from github.com)

For instance en-US or pt-BR

For instance `en-US` or `pt-BR`
Owner
I think they should be because of https://github.com/Plume-org/rocket_i18n/blob/master/src/with_rocket.rs#L24 and https://github.com/Plume-org/rocket_i18n/blob/master/src/with_actix.rs#L50 ? Or did you mean somewhere else?
elegaanz commented 5 years ago (Migrated from github.com)

Yes, but my concern was more about making sure they are working with gettext-macros too, and that the two-parts version is correctly used when needed if the one-part version is also available (not sure if I'm clear, sorry).

Yes, but my concern was more about making sure they are working with `gettext-macros` too, and that the two-parts version is correctly used when needed if the one-part version is also available (not sure if I'm clear, sorry).
Valodim commented 5 years ago (Migrated from github.com)

If I'm reading that code right, the locale part (so -Lo out of ln-Lo) is simply ignored?

I believe it is very important to support this because of the Chinese language. The locales "Chinese (Traditional)" and "Chinese (Simplified)" are quite different, but share the zh language prefix.

If I'm reading that code right, the locale part (so `-Lo` out of `ln-Lo`) is simply ignored? I believe it is very important to support this because of the Chinese language. The locales "Chinese (Traditional)" and "Chinese (Simplified)" are quite different, but share the `zh` language prefix.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: plume/rocket_i18n#17
Loading…
There is no content yet.