Conversation
- ChainInput from config json - Create route from backend to get Properties from config json - ResolvedProperties (WIP) Reviewed-by:andriac [Refs_ticket]: #40
- Solve problem to "keep" value when we are using chainInput - resetObject is adapt to new Form Reviewed-by: andriac [Refs_ticket]: #53
-"keep" property in specific config from types_site works with chainInput Reviewed-by: andriac [Refs_ticket]: #53
mvergez
left a comment
mvergez
left a comment
There was a problem hiding this comment.
Quelques changements et remarques :)
| methods=["GET"]) | ||
| def get_config_sites(module_code, object_type, id): | ||
| obj = get_config_object(module_code, object_type, id) | ||
| return obj['properties'] |
There was a problem hiding this comment.
Plutôt obj.get('properties', {}) ? Ou on veut que ça plante si l'obj n'a pas de properties ? Ou alors properties est initialisé à {}
There was a problem hiding this comment.
En l'écrivant je n'avais pas pensé au retour d'erreur possible. Effectivement ça peut être une possibilité de renvoyer une erreur , c'est plus un choix d'expérience utilisateur. Est ce que si l'utilisateur renseigne mal ses fichiers de config json , on renvoie une erreur où est ce qu'on veut que ça passe avec des properties vide .
There was a problem hiding this comment.
Oui je suis d'accord... A voir si cette route n'est pas appelée autre part où il n'y aurait pas obj['properties']. Je ne crois pas mais à vérifier quand même
There was a problem hiding this comment.
Cette route est appelée uniquement dans le component monitorin-form.component-.ts pour charger la config , du coup on laisse le obj[properties] ?
Le cas où "properties" est vide était un cas qui pouvait déjà arrivrer avant. Si une erreur survient elle surviendra pas à ce niveau là du code . Elle surviendra avant dans le fichier qui était déjà présent avant serializer.py :
| 'chain_show' | ||
| ); | ||
| if (chainShow) { | ||
| this.chainShow.push(chainShow.map((key) => this.obj.resolvedProperties[key])); |
There was a problem hiding this comment.
Apparemment, dans le problème remonté dans le sites_group cela viendrait de là...
There was a problem hiding this comment.
Quand tu dis le problème remonté dans le "sites_group" , tu parles de quel problème ? Merci
There was a problem hiding this comment.
Pardon, je crois qu'il y avait un problème d'enchaînement des saisies dans les groupes de sites
There was a problem hiding this comment.
Je viens de tester et j'arrive bien à enchainer les saisies de groupe de site avec le commentaire qui reste (car j'ai rajouté 'comment' à 'keep' dans le fichier sites_group.json). Tu arriverais à te souvenir à quel niveau il y aurait le problème ?
frontend/app/components/monitoring-form-g/monitoring-form.component-g.ts
Outdated
Show resolved
Hide resolved
frontend/app/components/monitoring-form-g/monitoring-form.component-g.ts
Outdated
Show resolved
Hide resolved
| } | ||
|
|
||
| const schema = this.schema(obj.moduleCode, obj.objectType); | ||
| for (const attribut_name of Object.keys(schema)) { |
There was a problem hiding this comment.
Le foreach est mieux je crois en javascript
There was a problem hiding this comment.
J'ai lu un peu de documentation à ce sujet et en fait en terme de performance c'est pas forcément le cas par contre il est dit que ça rend le code plus concis. A voir !
Source :
Reviewed-by: andriac
2 participants
Currently the chainInput Icon is "True" when "chained" is present in config.json
TODO : in other PR , "resolvedProperty" is not working currently cause of the way is set in backend . (See TODO in this PR)
Closes #53