fix(import): make config default_value_mapping multi-dest#3801
fix(import): make config default_value_mapping multi-dest#3801VincentCauchois wants to merge 1 commit intodevelopfrom
default_value_mapping multi-dest#3801Conversation
afb91ac to
2e21623
Compare
default_value_mapping multi-dest
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3801 +/- ##
===========================================
- Coverage 86.21% 86.20% -0.02%
===========================================
Files 141 141
Lines 10933 10935 +2
===========================================
Hits 9426 9426
- Misses 1507 1509 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
OK pour moi avec les ID par défaut. C'était déjà comme ça, y a pas d'autre solution simple et c'est une fonctionnalité peu (pas ?) utilisée. Et c'est plutôt sur ce point que je m'interroge. Je ne pense pas que la possibilité de masquer l'étape de mapping des nomenclatures soit vraiment utilisé et je le déconseille. Une partie de récapitulatif sur le fonctionnement de cette partie est disponible sur PnX-SI/gn_module_import#68 (comment). |
Replace GN [IMPORT] config param `DEFAULT_VALUE_MAPPING_ID` - single ID used for whatever destination - with `DEFAULT_VALUE_MAPPINGS`. This allows to specify one default mapping for each destination, which is necessary as a mapping should be destination-wise and is actually associated to one destination at most.
2e21623 to
f9b4852
Compare
Replace GN [IMPORT] config param
DEFAULT_VALUE_MAPPING_ID- single ID used for whatever destination - withDEFAULT_VALUE_MAPPINGS. This allows to specify one default mapping for each destination, which is necessary as a mapping should be destination-wise and is actually associated to one destination at most.