feature request #6619
Integrate Excel distribution import into taxeditor
Now the import of updated distribution data is finished we have to integrate it into the taxeditor
ref #6619: add the possibility to ignore methods of the configurator in the configuration wizard of imports and exports
#6 Updated by Andreas Müller over 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
The wizard can still be improved:
- The import should show up only if "show experimental features" is switched on. This is because PresenceAbsence terms are still hardcoded and therefore the import is not really usable for standard users.
We should not show "setIgnoreNul", "setInteractWithUser" and "setUseClassification" as they are not relevant for the distribution import. Remove selection of nomenclatural code (general it is not needed and with 7854fc266b I also fixed the NPE that comes if no code is defined) On the last page it says "Choose NormalExplicit" and also the subtitle is wrong (also always try to avoid camel case in UI). The selection box for area vocabulary is missing a label such as "Area vocabulary" Last but not least: no feedback is given at all, at least when I run it in an unvalid condition (I used it with FauEu data on my own test database which has no FauEu data). The distribution update import already gives a lot of feedback via "ImportResult". This should be shown at the end. E.g. via
you should get a complete report of what happened.
A general issue:
The subtitle "Configure the import mechanism" should be replaced by a specific label like "Configure Excel distribution update" (this is a general issue for all imports), "import mechanism" is generally not a good term
#7 Updated by Katja Luther over 3 years ago
- Status changed from Feedback to Resolved
Applied in changeset taxeditor|9313d25a2e00e474498e2713f4cb0b44245b37b1.