Bevinding #34753
closedNederlandse taal wordt niet correct geïnitialiseerd
100%
Description
Soms bij het starten van de app wordt de Nederlandse taal niet correct geïnitialiseerd.
Updated by Diego Mirandola 6 months ago
- Assignee changed from Diego Mirandola to Nume Groenewegen
Updated by Diego Mirandola 6 months ago
· Edited
Het probleem lijkt te zijn een race condition tussen de volgende twee flows:
- https://github.com/Alfresco/alfresco-ng2-components/blob/develop/lib/core/src/lib/translation/translation.service.ts#L68
- module X importeren
- translations voor module X ophalen in de huidige taal van de app, let op: bij het starten van de app, dit is altijd Engels https://github.com/Alfresco/alfresco-ng2-components/blob/develop/lib/core/src/lib/translation/translation.service.ts#L64
- taal selecteren, let op: dezelfde taal als in de vorige stap, dus het is tenminste 1 keer Engels per module
- https://github.com/Alfresco/alfresco-ng2-components/blob/develop/lib/core/src/lib/translation/translation.service.ts#L74
- app.config laden
- taal lezen vanuit gebruiker-preferences
- taal selecteren, let op: dit is de correcte taal
Als 2.3 gebeurt tussen 1.2 en 1.3 dan wordt de correcte taal overgeschreven door Engels.
De oorzaak van het probleem lijkt de aantal modules die geïmporteerd worden. Inderdaad, zien we het probleem vaker voorkomen in Tezza dan in contezza-reports en helemaal nooit in de default ACA.
Om dit te voorkomen, gaan we het ophalen van de translations en de selectie van de taal helemaal loskoppelen, dat is stap 1.3 verwijderen. Dit betekent dat de taal wordt altijd en alleen geselecteerd door de tweede flow, en dus op basis van app.config en gebruiker-preferences.
Updated by Diego Mirandola 6 months ago
- Status changed from In Progress to Ready in Test
Dit is in branch fix/translations/#34753
Graag testen op dev-tezza in verschillende apps.
Tot nu toe zijn tezza, contezza-reports en contezza-admin-tools gebuild met deze wijziging.
Updated by Nume Groenewegen 6 months ago
- Status changed from Ready in Test to Test approved
Getest voor apps tezza, contezza-search, contezza-admin-tools, contezza-reports, contezza-akten en vlisco-product-development.
De taal wordt steeds correct geladen, ook na meerdere keren uit en inloggen.
Apps doorgelopen op vertalingen en ziet er overal goed uit.
Kan mee met de release.
Updated by Diego Mirandola 6 months ago
- Status changed from Test approved to Resolved
- % Done changed from 0 to 100
In contezza-apps versie 4.2.2.
Updated by Rick de Rooij 6 months ago
Ticket bij Hyland support aangemaakt: #02025477
Updated by Diego Mirandola 2 months ago
Toegevoegd als bekend probleem in contezza-apps release notes.
