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
| // 26.StatisticalValidationMenu.tsx | ||
| 'statValidationMenu.bestFeatures': 'Most significant features', | ||
| 'statValidationMenu.bestFeatures.info': 'Most significant features for the survival analysis', | ||
| 'statValidationMenu.kaplanMeier': 'Kaplan-Meier', |
There was a problem hiding this comment.
Eliminar, es un nombre propio y no debería recibir traducciones (verificar si se está utilizando o no)
| 'statValidationMenu.bestFeatures': 'Most significant features', | ||
| 'statValidationMenu.bestFeatures.info': 'Most significant features for the survival analysis', | ||
| 'statValidationMenu.kaplanMeier': 'Kaplan-Meier', | ||
| 'statValidationMenu.kaplanMeier.info': 'curve showing survival or hazard ratio', |
There was a problem hiding this comment.
Ojo, acá no se está mencionando Kaplan-Meier. Debería decir 'Kaplan-Meier curve showing survival ... porque si no quedaría el texto incompleto a como estaba antes
| {/* Model details. */} | ||
| <Segment> | ||
| <Header as='h2' dividing>Model details</Header> | ||
| <Header as='h2' dividing>{intl.formatMessage({ id: 'common.details' }) + ' ' + intl.formatMessage({ id: 'statValidationMetrics.header.model' })}</Header> |
There was a problem hiding this comment.
No hace falta modularizar tanto. Por más que un texto tenga palabras que son comunes, no siempre hay que descomponerlas completamente. Acá si seleccionamos español por ejemplo va a aparecer "Modelo detalles" en vez de "Detalles del modelo" que sería la traducción correcta! Dejar solo el 'statValidationMetrics.header.model' y poner el texto que corresponde
2 participants
No description provided.