Feature/clinical options columns #52
Conversation
… into feature/clinical_options_columns
| }).catch((err) => { | ||
| console.log('Error parsing JSON ->', err) | ||
| console.log('Error getting user experiments', err) |
There was a problem hiding this comment.
Ya que estamos, corregir el mensaje de error. Acá no está trayendo los experimentos del usuario
There was a problem hiding this comment.
Dejar el comentario que editaste como estaba antes 'Error parsing JSON ->'. Ese estaba bien! Lo que está mal es el de abajo que dice "Error getting user experiments" cuando en realidad choca contra un endpoint que compara los Samples en común. Debería decir algo como "Error getting samples in common"
| @@ -60,12 +60,13 @@ interface NewFileFormProps { | |||
| isEditing: boolean, | |||
| uploadState: Nullable<UploadState> | |||
| uploadFile: () => void, | |||
| fileChange: (e: any) => void, | |||
| fileChange: any, // (e: any) => void | |||
There was a problem hiding this comment.
JAJAJA, recuerdo poner ese any, para probar sin problemas. Ahi lo vuelvo como estaba antes, un (e:any)=> void. Porque recibe almenos un metodo que va a recibir el evento, no un cualquier cosa
| @@ -71,6 +71,8 @@ interface ClinicalSourceState { | |||
| cgdsStudyName: Nullable<string>, | |||
| /** alert interface */ | |||
| alert: CustomAlert, | |||
| /** posibles values for survival tuple */ | |||
There was a problem hiding this comment.
Por consistencia, arrancá los comentarios en mayúscula hdp!
There was a problem hiding this comment.
Vos sabes que nunca me di cuenta ajajaj. Dale ahora lo tomo para toda la documentacion que vaya a hacer. Gracias!
| @@ -20,6 +20,8 @@ interface SurvivalTuplesFormProps { | |||
| handleSurvivalFormDatasetChanges: (idx: number, name: string, value) => void, | |||
| addSurvivalFormTuple: () => void, | |||
| removeSurvivalFormTuple: (idx: number) => void | |||
| /** posibles values for survival tuple if undefined allows the users to write their own text */ | |||
There was a problem hiding this comment.
Idem comentario de más arriba, que arranque con mayúscula el mensaje, así queda igual en toda la plataforma
| @@ -57,12 +59,31 @@ export const SurvivalTuplesForm = (props: SurvivalTuplesFormProps) => { | |||
|
|
|||
| const addSurvivalFormTuple = props.addSurvivalFormTuple | |||
|
|
|||
| /** | |||
| * method to handle select change. | |||
| resolve([]) | ||
| } | ||
|
|
||
| fileReader.onerror = (event) => reject(event) |
There was a problem hiding this comment.
Acá simplemente se puede hacer fileReader.onerror = reject
There was a problem hiding this comment.
Dale, perfecto! Aprovecho y lo añado a la función que busca todos los headers del archivo.
… into feature/clinical_options_columns t :wq :# especially if it merges an updated upstream into a topic branch.
| alert: CustomAlert, | ||
| /** posibles values for survival tuple */ | ||
| /** Posibles values for survival tuple */ |
There was a problem hiding this comment.
"Possible" en singular y con doble "s"! Hay una extensión en VSCode que se llama Code Spell Checker que me salva la vida en el laburo todos los días con la ortografía en inglés:
| @@ -20,7 +20,7 @@ interface SurvivalTuplesFormProps { | |||
| handleSurvivalFormDatasetChanges: (idx: number, name: string, value) => void, | |||
| addSurvivalFormTuple: () => void, | |||
| removeSurvivalFormTuple: (idx: number) => void | |||
| /** posibles values for survival tuple if undefined allows the users to write their own text */ | |||
| /** Posibles values for survival tuple if undefined allows the users to write their own text */ | |||
There was a problem hiding this comment.
Mismo comentario que arriba sobre la falta
#Fix select from file in survival tuples form