Add new contingency list by filter #711
Conversation
basseche
changed the title
Signed-off-by: basseche <[email protected]>
Signed-off-by: basseche <[email protected]>
Signed-off-by: basseche <[email protected]>
basseche
changed the title
| @@ -42,7 +42,7 @@ export default function ContingencyListCreationForm() { | |||
| const contingencyListTypeField = ( | |||
| <RadioInput | |||
| name={FieldConstants.CONTINGENCY_LIST_TYPE} | |||
| options={Object.values(ContingencyListType)} | |||
| options={[ContingencyListType.CRITERIA_BASED, ContingencyListType.EXPLICIT_NAMING]} | |||
There was a problem hiding this comment.
C'est exactement la même chose non , pourquoi avoir changé ?
| id?: UUID; | ||
| } | ||
|
|
||
| const schema: any = yup.object().shape({ |
There was a problem hiding this comment.
Si c'est parce que tu as un problème type script, vu qu'il suffit d'importer la bonne version de common ui pour régler le problème, tu peux l'enlever
There was a problem hiding this comment.
Il y a une faute d'orthographe c'est form et pas from
|
|
||
| const equipmentTypes: string[] = useMemo(() => { | ||
| return [ | ||
| EquipmentType.TWO_WINDINGS_TRANSFORMER, |
There was a problem hiding this comment.
À propos de EquipmentType
Deprecated:
move to a specialized subtype as this one is used everywhere and cause problems
| const SEPARATOR_TYPE = 'SEPARATOR'; | ||
| const attributes: IdentifiableAttributes[] = response.equipmentIds.map( | ||
| (element: IdentifiableAttributes) => { | ||
| const equipmentType: string = getBasicEquipmentLabel(element?.type) ?? null; |
There was a problem hiding this comment.
A quoi sert ?? null a la fin ?
| if (response.notFoundIds?.length > 0) { | ||
| attributes.push({ id: SEPARATOR_TYPE, type: '' }); | ||
| response.notFoundIds.forEach((element: IdentifiableAttributes) => { | ||
| const equipmentType: string = getBasicEquipmentLabel(element?.type) ?? null; |
There was a problem hiding this comment.
Même remarque qu'en haut
| return map; | ||
| }, []); | ||
|
|
||
| const onStudyChanged = useCallback(() => { |
There was a problem hiding this comment.
Call back qui déclenche un rendu supplémentaire inutile, autant intégrer directement le code dans le useEffect plus bas
needs changes from commons-ui to compile :
gridsuite/commons-ui#839