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

fix: notification to channel language #1133

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

fix: notification to channel language #1133

wants to merge 17 commits into from

Conversation

SputNikPlop
Copy link
Collaborator

This aims to resolve #919
I need this looked over first also probably want to update all the translation messages in this PR.

@kevmo314
Copy link
Contributor

What does twitch translate the tiers as? It should match the labeling they use since they are more like proper nouns.

@SputNikPlop
Copy link
Collaborator Author

What does twitch translate the tiers as? It should match the labeling they use since they are more like proper nouns.

muxfd subscribed at Tier 1. They've subscribed for 10 months! or muxfd subscribed at Tier 1. They've subscribed for 10 months, currently on a 10 month streak! Should probably also think about exposing the streak amount maybe as well.

TextSpan(
text: model.total > 1 ? "subscriptions. " : "subscription. "),
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you know what this part of the logic was I don't remember off the top of my head, would like to make the translation part more descriptive for our translators.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This won't work with translations as some languages don't let you composite like this. See https://docs.flutter.dev/ui/accessibility-and-internationalization/internationalization#placeholders-plurals-and-selects

We'll need to bake the pluralization into the translation string itself.

@SputNikPlop
Copy link
Collaborator Author

The subscription message I will investigate into as a separate PR.

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.

[chat_history] switch notification messages to use the language of the channel
2 participants