nextcloud / unsplash

πŸ“ΈπŸ”€β˜οΈ Random Nextcloud log in background from Unsplash
https://apps.nextcloud.com/apps/unsplash
GNU Affero General Public License v3.0
92 stars 23 forks source link

l10n: Splitted text strings confusing translators #136

Closed rakekniven closed 1 month ago

rakekniven commented 1 month ago

On Transifex there is no way to get the relation of part strings like

https://github.com/nextcloud/unsplash/blob/69fe57a2ff60ab82fdf8cfd4e5c529017b6b9b11/templates/settings/admin.php#L91

Please improve.

Reported at Transifex.

rakekniven commented 1 month ago

@newhinton Blame brought me to you.

rakekniven commented 1 month ago

"You should not need to apply for production"

Context please. Is this license related?

newhinton commented 1 month ago

Oh, yeah sorry. I have improved the string so that it is more or less self-contained.

"You should not need to apply for production"

The linked page asks the user to apply for production usage of their newly generated api token. For this apps's usage, this is not really required since we access their api only once a day, which is squarely in their 50hits/h limit for a demo application.

I am not sure if i can provide more context here, since the context is on an external site.

rakekniven commented 1 month ago

You can add context for translators directly to your code. See https://docs.nextcloud.com/server/latest/developer_manual/basics/front-end/l10n.html#provide-context-hints-for-translators for details.