Implementation of Tags in Add and Edit Biomarkers and add tags panel in Biomarkers#96
Conversation
| newFile: NewFile, | ||
| tagOptions: DropdownItemProps[], | ||
| tags: DjangoTag[], | ||
| uploadingFile: boolean, |
There was a problem hiding this comment.
Hay una banda de props acá que no se usan en todo el componente, eliminar para simplificar acá y en los demás componentes que están pasándose props en cadena hasta llegar hasta acá
| * Removes a Survival data tuple for a CGDSDataset | ||
| * @param idxSurvivalTuple Index in survival tuple | ||
| */ | ||
| removeSurvivalFormTuple = (idxSurvivalTuple: number) => { |
There was a problem hiding this comment.
Eliminar este y las demás funciones que no tienen que ver. Resolvería primero el comentario sobre eliminar las props innecesarias en el componente NewBiomarkerForm.tsx. Con eso vas a ver que empiezan a aparecer un montón de campos del estado como newFile que no sirven para nada
| dataset.survival_columns[idxSurvivalTuple][name] = value | ||
| this.setState({ newBiomarker }) | ||
| } | ||
| handleSurvivalFormDatasetChanges = (idxSurvivalTuple: number, name: string, value: any) => { |
There was a problem hiding this comment.
Exactamente lo mismo que el comentario anterior, acá se manejan cosas como archivos para enviar al server, no se requieren acá. Están porque las necesitás para pasarlas como props, al resolver ese problema en el componente NewBiomarkerForm.tsx va a salir a la luz todo lo que sobra para poder eliminarlo
| /** | ||
| * Fetches the Institutions of which the User is part of | ||
| */ | ||
| getUserInstitutions () { |
There was a problem hiding this comment.
Eliminar todo lo referente a las instituciones, no se usa en ningún lado
There was a problem hiding this comment.
Ver los errores del IDE, está tirando un warning de que esta constante no se está usando:
| tags: DjangoTag[] | ||
| uploadingFile: boolean | ||
| handleAddFileInputsChange: (name: string, value: any) => void | ||
| newFile: NewFile |
There was a problem hiding this comment.
Mismo problema, no se está usando en todo el componente. Hacer Ctrl + F con todas las nuevas props y ver si se están usando o no y limpiar
2 participants
No description provided.