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

Russian translation & Fixes of non-localized strings #794

Merged
merged 6 commits into from
Jun 21, 2023

Conversation

Snow4DV
Copy link
Contributor

@Snow4DV Snow4DV commented Jun 21, 2023

Translated app to russian and fixed some unlocalized string here and there

<string name="inbox_activity_replies">Replies</string>
<string name="inbox_activity_mentions">Mentions</string>
<string name="inbox_activity_messages">Messages</string>
<string name="bookmarks_activity_saved">Saved</string>
Copy link
Collaborator

Choose a reason for hiding this comment

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

There is already two Saved string resources, maybe use an existing? Or is it not applicable in this context?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably. It is applicable in this context but there already were duplicating string resources previously for different contexts so i thought it would be better to separate them to leave some flexibility.

Copy link
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

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

Thanks, I'm sure there's more that we're missing too, but we can find them later.

Run ./gradlew formatKotlin to pass lint.

@@ -103,7 +103,7 @@ fun PersonProfileActivity(
PersonProfileHeader(
scrollBehavior = scrollBehavior,
personName = if (savedMode) {
"Saved"
ctx.getString(R.string.bookmarks_activity_saved)
Copy link
Member

Choose a reason for hiding this comment

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

Good catch, thanks.

@Snow4DV
Copy link
Contributor Author

Snow4DV commented Jun 21, 2023

Done, now it passes the check

@dessalines dessalines enabled auto-merge (squash) June 21, 2023 14:36
@dessalines dessalines merged commit 30aa860 into LemmyNet:main Jun 21, 2023
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