[IMPORT] Feat context parameter import #3277
Conversation
andriacap
force-pushed
the
feat/context-parameter-import
branch
2 times, most recently
from
b4efac7 to
ef99612
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/import-monitorings #3277 +/- ##
========================================================
Coverage 83.94% 83.95%
========================================================
Files 122 122
Lines 9721 9727 +6
========================================================
+ Hits 8160 8166 +6
Misses 1561 1561
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
andriacap
force-pushed
the
feat/context-parameter-import
branch
2 times, most recently
from
554881c to
aca2da1
Compare
andriacap
force-pushed
the
feat/context-parameter-import
branch
from
aca2da1 to
164e01e
Compare
jacquesfize
force-pushed
the
feat/import-monitorings
branch
from
2b5f26f to
dcc764e
Compare
- Format params to FieldsMapping interface - Send on upload route frontend - Set fieldsmapping for t_import on upload route backend Reviewed-by: andriacap
- Synthese test frontend (ok) - wip : make occhab condition and refact test Reviewed-by: andriacap
Use same attribute for QA in all elements in fields mapping step Reviewed-by: andriacap
Reviewed-by: andriacap
- Change fieldmapping for sample data - Add delete command to each test for fieldmapping context Reviewed-by: andriacap
Reviewed-by: andriacap
andriacap
force-pushed
the
feat/context-parameter-import
branch
from
3956690 to
85dca11
Compare
- Test upload_route with context parameter Reviewed-by: andriacap
|
J'ai testé à la main et tout a l'air de fonctionner correctement, juste pour les tests frontend, j'ai l'impression que les erreurs viennent de conflit dans les data-qa, je pense que c'est parce que certain champ sont à la fois dans les habitats et dans les stations, dans ce cas le champ existe deux fois, ce qui fait que deux champs avec le même data-qa existe et ça fait planter les tests. |
| @@ -91,9 +104,9 @@ export class UploadFileStepComponent implements OnInit { | |||
| } | |||
| onSaveData(): Observable<Import> { | |||
There was a problem hiding this comment.
Attention, onSaveData n'est exécuté seulement si le formulaire est modifié ! par conséquent, si je reviens en arrière pour modifier uniquement les valeurs dans les queryParams, cela ne fonctionnera pas ...
TACHES PROJET IMPORT :
feat(wip): get context import by query params url
Format params to FieldsMapping interface
Send on upload route frontend
Set fieldsmapping for t_import on upload route backend
Test backend
Test frontend : un test e2e a été ajouté pour tester la fonctionnalité
Tests frontend existants : certains sont fonctionnels mais ne sont pas lié aux développements de cette fonctionnalité (à résoudre dans une autre PR ?)
Reviewed-by: andriacap
Tâche(s) dépendante(s)