Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: FLUI-106 add locales files from cqdg #369

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

atoulous
Copy link
Contributor

feat: FLUI-106 add locales files from cqdg

  • closes #TICKET_NUMBER

Description

https://ferlab-crsj.atlassian.net/browse/FLUI-106

Copy link

github-actions bot commented Jan 17, 2024

Coverage report for packages/ui

St.
Category Percentage Covered / Total
🟢 Statements 100% 0/0
🟢 Branches 100% 0/0
🟢 Functions 100% 0/0
🟢 Lines 100% 0/0

Test suite run success

0 tests passing in 0 suite.

Report generated by 🧪jest coverage report action from 96b2b98

Copy link

github-actions bot commented Jan 17, 2024

Project Coverage and Test

Statements : 26.92% ( 4693/17428 )
Branches : 51.49% ( 361/701 )
Functions : 27.91% ( 103/369 )
Lines : 26.92% ( 4693/17428 )

Test Suites: 29 passed, 29 total
Tests: 131 passed, 131 total
Snapshots: 0 total
Time: 51.865 s
Ran all test suites.

import en from './en';
import fr from './fr';

export const ageCategories = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Juste une question c'est dans tous les projets que l'on a besoin de ça ? je ne vois pas dans include. Et c'est dans locales mais il n'est qu'en anglais.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non c'est que dans cqdg pour le moment mais ca pourrait etre utilisé pour les catégories d'age dans les autres projets aussi.
Oui la value que ca donne ce n'est qu'en anglais car on ne traduit pas les valeurs des données pour rester consistant

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

et autre question pourquoi avoir besoin d'une méthode pour transformer ton tableau ? On ne peut pas avoir directement l'objet que tu veux ? Ou tu utilises ailleurs le tableau ageCategories dans CQDG en plus du format retourné par la méthode ?

@ethienneroy
Copy link
Contributor

est-ce vraiment le role de UI d'avoir les traductions ? si on garde dans l'idee qu'on veut creer une lib pour extraires les autres fonction communes ?

@atoulous
Copy link
Contributor Author

est-ce vraiment le role de UI d'avoir les traductions ? si on garde dans l'idee qu'on veut creer une lib pour extraires les autres fonction communes ?

Le role de ferlab-ui ? Oui c'est ce qu'on veut expérimenter justement pour centraliser nos traductions.
Ca n'empêche pas chaque projet d'overrider les clefs en fonction du besoin.
On pourrait ensuite imaginer un autre service plus travaillé si le besoin évolue, avec par exemple une interface cloud pour modifier les traductions.

@ethienneroy
Copy link
Contributor

est-ce vraiment le role de UI d'avoir les traductions ? si on garde dans l'idee qu'on veut creer une lib pour extraires les autres fonction communes ?

Le role de ferlab-ui ? Oui c'est ce qu'on veut expérimenter justement pour centraliser nos traductions. Ca n'empêche pas chaque projet d'overrider les clefs en fonction du besoin. On pourrait ensuite imaginer un autre service plus travaillé si le besoin évolue, avec par exemple une interface cloud pour modifier les traductions.

on pourrait discuter de ca justement car on pourrait rendre UI agnostique des traductions et c'est le projet qui load ferlab avec un fichier de traductions qui est extrait ailleur 🤔

@francisl
Copy link
Collaborator

Nous allons faire une autre rencontre de discussion, je crois qu'ils y a plusieurs points en questionnement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants