ioBroker.admin icon indicating copy to clipboard operation
ioBroker.admin copied to clipboard

Translations update from ioBroker Translation System

Open ioBrokerTranslator opened this issue 1 year ago • 7 comments

Translations update from ioBroker Translation System for ioBroker Adapters/admin.

It also includes following components:

Current translation status:

Weblate translation status

ioBrokerTranslator avatar Feb 19 '24 09:02 ioBrokerTranslator

@Apollon77

Not sure why this PR has been created. I definitly did NOT change anything at weblate within the last hours. Last action was adding solaredge - three days ago. So I have no clue why weblate lists those old commits again ...

Maybe this is caused by updates done externally (= new text added due to license infos). Those additions change the complete german file due to diffenrences in whitespaces if I'm corre ct reading that commit.

mcm1957 avatar Feb 19 '24 09:02 mcm1957

Probably really a problem with the squash merges..

foxriver76 avatar Feb 19 '24 10:02 foxriver76

would this maybe help us @Apollon77 https://docs.weblate.org/en/latest/admin/addons.html#squash-git-commits

I have never looked into weblate tbh

foxriver76 avatar Feb 19 '24 10:02 foxriver76

Looks like that addon will sqash outgoing commits (commits issued from weblate). As this will delete the change log inside the PR I'm not sure wether this will be a desired feature.

mcm1957 avatar Feb 19 '24 10:02 mcm1957

hm expected that.. then we would need to activate normal merge commits at least for weblate PRs

foxriver76 avatar Feb 19 '24 10:02 foxriver76

I'm still not sure wethe rthe squash merge is the main problem.

Currently admin is locke due to merge conflict. I'n not sure whether the first weblate PR has been merger BEFORE the changes done by license translations.

Maybe the problem is only caused by the fact that admin changes whitespaces in an incompatible way during translation. Normal nopm run translate does not seem to do this. So mayby the admin tool used fpor translations could be adjusted to matching indention and this would fix it too.

mcm1957 avatar Feb 19 '24 10:02 mcm1957

okay fyi I also merged a noop commit previously that day

foxriver76 avatar Feb 19 '24 10:02 foxriver76