Skip to content

Missing translations master#2908

Merged
lslezak merged 12 commits intomasterfrom
missing_translations_master
Nov 24, 2025
Merged

Missing translations master#2908
lslezak merged 12 commits intomasterfrom
missing_translations_master

Conversation

@lslezak
Copy link
Copy Markdown
Contributor

@lslezak lslezak commented Nov 24, 2025

Just merging the SLE-16 branch to master, includes fixed translations #2904

ancorgs and others added 12 commits November 7, 2025 16:08
## Problem

During some manual testing, @tacerus found a pretty serious bug in the
Agama storage proposal.

When distributing several partitions over several new MD RAIDs, it may
happen that Agama locates some of those partitions in the WRONG RAID
DEVICE!

That happened because Agama tried to optimize the size distribution of
those partitions, but in the process it ignored a very basic constraint:
the exact RAID in which those partitions should be created.

## Solution

Honor the relationship between new MDs and its partitions.

## Testing

Unit tests included
Agama-weblate commit: 4bcf40feaa6d02ef3389db87d81a9b6a6185ec1c
Updating the web translation files from the agama-weblate repository
Agama-weblate commit: 730f2f7dcc4d1715cffbd7a633c63901b636f290
Updating the web translation files from the agama-weblate repository
## Problem

- Some missing translations in the web UI
- https://bugzilla.suse.com/show_bug.cgi?id=1250059#c0

## Details

The code at the top level like

```
const message = _("foo")
```
does not work properly because the code is evaluated too early, even
before the component is actually displayed and before the translations
are loaded from the server.

## Solution

There are basically two solutions:

1. Use `N_()` function instead of `_()` at the top level and then use
`_()` when using the constant later.
2. Move the constant into a function (or function component) which is
called (and evaluated/retranslated) in every rendering of the component.

## Testing

- Tested manually.
- Because most of the problems were in special storage technologies like
iSCSI or DASD I was able to test only the error message in the software.
And even that was a bit tricky, to trigger the error I configured an
invalid installation repository with the
`inst.instal_url=http://example.com` boot parameter.
- I also verified the generated POT file after this change to ensure no
translation was lost. The generated POT is the very same (except the
line numbers in comments) as before the change.

## Additional changes

- Enabled eslint check which finds the top level translations to avoid
similar problems in the future
- Use NodeJS v22 in the CI build, that's the same version as in SLE-16.
Moreover v18 is obsolete and out of support for several months.
:open_mouth:

## Screenshots

Original SLE-16 system without the patch:
<img width="1280" height="800" alt="agama-missing-translation-software"
src="https://github.com/user-attachments/assets/2244b15e-3c5f-486d-b118-d5c84a7bdb0f"
/>

Fixed system after applying this patch:
<img width="1280" height="800"
alt="agama-missing-translation-software-fixed"
src="https://github.com/user-attachments/assets/7865c79e-7868-4614-8146-2f8da20f3ffe"
/>

---------

Co-authored-by: Martin Vidner <mvidner@suse.com>
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 64.147% (+0.002%) from 64.145%
when pulling be0bd06 on missing_translations_master
into 87a04f1 on master.

@lslezak lslezak merged commit 116ab0b into master Nov 24, 2025
10 checks passed
@lslezak lslezak deleted the missing_translations_master branch November 24, 2025 16:20
imobachgs added a commit that referenced this pull request Jan 10, 2026
Merge the new HTTP API. Each PR has been already reviewed, so it should
be safe to merge it.

* #1829
* #2508
* #2772
* #2826
* #2848
* #2860
* #2863
* #2866
* #2867
* #2869
* #2870
* #2871
* #2872
* #2873
* #2874
* #2875
* #2876
* #2877
* #2880
* #2881
* #2882
* #2884
* #2885
* #2886
* #2891
* #2892
* #2893
* #2894
* #2895
* #2896
* #2897
* #2898
* #2899
* #2900
* #2901
* #2902
* #2903
* #2904
* #2908
* #2909
* #2910
* #2912
* #2913
* #2914
* #2915
* #2916
* #2917
* #2918
* #2920
* #2921
* #2923
* #2924
* #2926
* #2928
* #2929
* #2930
* #2933
* #2934
* #2935
* #2936
* #2938
* #2939
* #2942
* #2943
* #2944
* #2945
* #2946
* #2947
* #2948
* #2950
* #2951
* #2952
* #2954
* #2955
* #2956
* #2957
* #2958
* #2959
* #2960
* #2961
* #2963
* #2964
* #2965
* #2967
* #2968
* #2969
* #2970
* #2971
* #2972
* #2974
* #2975
* #2977
* #2978
* #2980
* #2982
* #2983
* #2984
* #2988
* #2989
* #2991
* #2992
* #2993
* #2994
* #2995
* #2996
* #2997
* #2999
@imobachgs imobachgs mentioned this pull request Mar 17, 2026
imobachgs added a commit that referenced this pull request Mar 17, 2026
Prepare to release version 19.

* #1829
* #2508
* #2772
* #2818
* #2826
* #2848
* #2860
* #2863
* #2864
* #2866
* #2867
* #2869
* #2870
* #2871
* #2872
* #2873
* #2874
* #2875
* #2876
* #2877
* #2880
* #2881
* #2882
* #2884
* #2885
* #2886
* #2891
* #2892
* #2893
* #2894
* #2895
* #2896
* #2897
* #2898
* #2899
* #2900
* #2901
* #2902
* #2903
* #2904
* #2908
* #2909
* #2910
* #2912
* #2913
* #2914
* #2915
* #2916
* #2917
* #2918
* #2920
* #2921
* #2923
* #2924
* #2926
* #2928
* #2929
* #2930
* #2933
* #2934
* #2935
* #2936
* #2937
* #2938
* #2939
* #2942
* #2943
* #2944
* #2945
* #2946
* #2947
* #2948
* #2949
* #2950
* #2951
* #2952
* #2954
* #2955
* #2956
* #2957
* #2958
* #2959
* #2960
* #2961
* #2963
* #2964
* #2965
* #2967
* #2968
* #2969
* #2970
* #2971
* #2972
* #2974
* #2975
* #2977
* #2978
* #2980
* #2981
* #2982
* #2983
* #2984
* #2988
* #2989
* #2990
* #2991
* #2992
* #2993
* #2994
* #2995
* #2996
* #2997
* #2998
* #2999
* #3000
* #3001
* #3002
* #3004
* #3005
* #3006
* #3007
* #3008
* #3009
* #3011
* #3012
* #3013
* #3014
* #3015
* #3016
* #3018
* #3019
* #3020
* #3021
* #3022
* #3023
* #3024
* #3025
* #3026
* #3027
* #3028
* #3029
* #3030
* #3031
* #3033
* #3034
* #3035
* #3036
* #3037
* #3039
* #3040
* #3041
* #3042
* #3043
* #3044
* #3045
* #3046
* #3047
* #3048
* #3049
* #3050
* #3051
* #3052
* #3053
* #3054
* #3055
* #3056
* #3057
* #3058
* #3060
* #3061
* #3062
* #3063
* #3064
* #3065
* #3066
* #3067
* #3068
* #3069
* #3070
* #3071
* #3072
* #3073
* #3074
* #3075
* #3076
* #3077
* #3078
* #3079
* #3086
* #3087
* #3088
* #3089
* #3090
* #3091
* #3092
* #3093
* #3094
* #3095
* #3096
* #3097
* #3098
* #3099
* #3100
* #3101
* #3102
* #3103
* #3104
* #3105
* #3106
* #3107
* #3108
* #3109
* #3110
* #3112
* #3113
* #3114
* #3115
* #3116
* #3117
* #3118
* #3119
* #3120
* #3122
* #3123
* #3124
* #3127
* #3128
* #3129
* #3130
* #3131
* #3133
* #3134
* #3135
* #3136
* #3137
* #3138
* #3139
* #3140
* #3141
* #3142
* #3143
* #3144
* #3145
* #3146
* #3147
* #3148
* #3149
* #3150
* #3151
* #3152
* #3153
* #3154
* #3155
* #3157
* #3158
* #3159
* #3160
* #3161
* #3162
* #3163
* #3164
* #3165
* #3166
* #3167
* #3168
* #3169
* #3170
* #3174
* #3175
* #3176
* #3177
* #3178
* #3179
* #3181
* #3182
* #3184
* #3185
* #3186
* #3188
* #3189
* #3190
* #3191
* #3192
* #3194
* #3195
* #3196
* #3197
* #3198
* #3199
* #3200
* #3201
* #3202
* #3203
* #3205
* #3206
* #3208
* #3209
* #3210
* #3213
* #3214
* #3215
* #3216
* #3217
* #3218
* #3219
* #3220
* #3222
* #3223
* #3224
* #3225
* #3226
* #3227
* #3228
* #3229
* #3230
* #3231
* #3232
* #3233
* #3234
* #3235
* #3236
* #3237
* #3238
* #3239
* #3240
* #3241
* #3242
* #3243
* #3244
* #3246
* #3247
* #3248
* #3250
* #3251
* #3252
* #3253
* #3254
* #3255
* #3256
* #3257
* #3258
* #3259
* #3260
* #3261
* #3262
* #3263
* #3265
* #3266
* #3267
* #3268
* #3269
* #3270
* #3271
* #3272
* #3273
* #3274
* #3275
* #3276
* #3277
* #3278
* #3279
* #3280
* #3281
* #3282
* #3283
* #3284
* #3285
* #3286
* #3287
* #3288
* #3289
* #3290
* #3291
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.

5 participants