search

nextcloud / android

📱 Nextcloud Android app
https://play.google.com/store/apps/details?id=com.nextcloud.client
GNU General Public License v2.0
4.18k stars 1.75k forks source link

i18n: Text strings for setting status #9750

Open Valdnet opened 2 years ago

Valdnet commented 2 years ago

The following text strings:

obraz

Should be changed to:

They are not present in the application code, which can be thought to be downloaded from Android system.

They are translated into Polish as:

Which contradicts the given function. It should be correct:

obraz

AlvaroBrey commented 2 years ago

@tobiasKaminsky do you know where these strings come from?

tobiasKaminsky commented 2 years ago

I think from here: https://github.com/nextcloud/android/blob/7db8069ff06e72e9a42831d485ae7deeaf34b5f6/src/main/java/com/owncloud/android/ui/adapter/PredefinedStatusViewHolder.kt#L46

simunde commented 2 years ago

If the issue is still persisting may I work on it ?

Valdnet commented 2 years ago

@simunde If you want to fix it, commit then add to PR and wait for approval 😉.

hmncube commented 1 year ago

@tobiasKaminsky I am working on this issue and when I change from In 1 hour to 1 hour the Polish translation will change from Za 1 godzinę to 1 godzinę not Przez 1 godzinę. Is this correct or I am missing something?

tobiasKaminsky commented 1 year ago

Not sure about polish translation. Maybe @Valdnet can help out?

Valdnet commented 1 year ago

The text string is most likely from Android. Not present in Transifex translation. I am in favor of the proposal given by @hmncube.

tobiasKaminsky commented 1 year ago

Fine with me. We have same in web ui.