Global translations #86
Conversation
| <List> | ||
| <List.Item icon='user' content='PhD Martín Abba' /> | ||
| <List.Item icon='user' content='Dr. MSc. Matias Butti' /> | ||
| <List.Item icon='user' content={intl.formatMessage({ id: 'about.coordinator.Abba' })} /> |
There was a problem hiding this comment.
Los nombres propios no se traducen, ya que no van a cambiar dependiendo del idioma, dejar todos los nombres que aparecen en el archivo como estaban antes (eliminando también su contenido en es.ts y en.ts)
| { name: intl.formatMessage({ id: 'inference.table.columns.model' }), serverCodeToSort: 'model', width: 1 }, | ||
| { name: intl.formatMessage({ id: 'inference.table.columns.date' }), serverCodeToSort: 'created' }, | ||
| { name: intl.formatMessage({ id: 'inference.table.columns.dataset' }) }, | ||
| { name: intl.formatMessage({ id: 'inference.table.columns.actions' }) } |
There was a problem hiding this comment.
- Que "Actions" sea también parte del
common(ej.:common.actions) en las traducciones, porque está en todas las tablas. - Reemplazar en todos los archivos usando esa de
common.actions - Eliminar las otras duplicaciones en los archivos es.ts y en.ts
| disabled={!formIsValid()} | ||
| > | ||
| Confirm | ||
| {intl.formatMessage({ id: 'inference.new.confirm' })} |
There was a problem hiding this comment.
Idem comentario anterior, pasar a una clave common.confirm y usar esa en todos lados
| headers={[ | ||
| { name: 'Sample', serverCodeToSort: 'sample', width: 3, textAlign: 'center' }, | ||
| { name: 'Cluster', serverCodeToSort: 'cluster', width: 2, textAlign: 'center' } | ||
| { name: intl.formatMessage({ id: 'inference.table.columns.sample' }), serverCodeToSort: 'sample', width: 3, textAlign: 'center' }, |
There was a problem hiding this comment.
Si acá se usa un ID para la palabra "Sample" entonces se puede usar el mismo ID abajo en vez de 'inference.table.search.label'. Así no duplicamos todo y llenamos de ruido con duplicaciones en los archivos de traducción
| headers={[ | ||
| { name: 'Sample', serverCodeToSort: 'sample', width: 3, textAlign: 'center' }, | ||
| { name: 'Predicted time', serverCodeToSort: 'prediction', width: 2, textAlign: 'center' } | ||
| { name: intl.formatMessage({ id: 'inference.timeTable.columns.sample' }), serverCodeToSort: 'sample', width: 3, textAlign: 'center' }, |
There was a problem hiding this comment.
Idem comentario anterior: las palabras que aparezcan en un mismo archivo se pueden compartir. Es más, si ves que la palabra "Sample" aparece 200 veces sentite con la libertad de ponerlo en un common y usar ese ID por todos lados!
| className='clickable' | ||
| color='blue' | ||
| title='Details' | ||
| title={intl.formatMessage({ id: 'moleculesTable.details' })} |
There was a problem hiding this comment.
"Details" ya lo ví varias veces, pasar a common
| value={form.name} | ||
| icon='asterisk' | ||
| placeholder='Name' | ||
| placeholder={intl.formatMessage({ id: 'newStatValidation.form.name' })} |
There was a problem hiding this comment.
Tanto "Name", como "Description" (con y sin el "(optional)") y "Required field" son cosas que aparecen en todos lados, mover a common y reemplazar en donde corresponda
| disabled={!selectedTrainedModelIsValid} | ||
| > | ||
| Continue | ||
| {intl.formatMessage({ id: 'newStatValidation.button.continue' })} |
There was a problem hiding this comment.
"Continue" ya lo ví varias veces, pasar a common
| onClick={() => props.setActiveItem(ActiveStatValidationsItemMenu.KAPLAN_MEIER)} | ||
| > | ||
| Kaplan-Meier | ||
| {intl.formatMessage({ id: 'statValidationMenu.kaplanMeier' })} |
There was a problem hiding this comment.
"Kaplan-Meier" es un nombre propio, no traducir. Abajo está bien porque está dentro de una oración, pero acá que aparece el nombre solo dejar como estaba
| @@ -0,0 +1,97 @@ | |||
| import React from 'react' | |||
| import { Container, Divider, Grid, Header, Segment, List } from 'semantic-ui-react' | |||
There was a problem hiding this comment.
Corregir todos los warnings que aparecen, chequear con npm run check-all o npm run check-lint. Se pueden corregir algunos automáticamente con npm run fix-lint si mal no recuerdo (está el comando en el archivo package.json)
No description provided.