Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSV Export und Import #1637

Open
5 tasks done
RBordoni opened this issue Nov 6, 2024 · 16 comments · May be fixed by #1785
Open
5 tasks done

CSV Export und Import #1637

RBordoni opened this issue Nov 6, 2024 · 16 comments · May be fixed by #1785
Assignees
Labels
enhancement New feature or request

Comments

@RBordoni
Copy link

RBordoni commented Nov 6, 2024

Export

  • CSV Export: Nur Stammdaten-Export, soll simpel gehalten werden. (Felder wir jetziger Import, Anpassung gemäss Kommentar unten)
  • TVD exportieren
  • Zusätzliche Ids exportieren (ID GeODIN , ID Kanton...)

Import

Weitere Angaben unter #1481

Anmerkungen:

  • Es ist mit dem UX-Team zu schauen, wie die Funktionen im UI platziert werden.
  • Attachments müssen im CSV Import/Export nicht berücksichtigt werden.
  • LV03 und LV95 bei Import und Export berücksichtigen.
@RBordoni RBordoni added the enhancement New feature or request label Nov 6, 2024
@RBordoni RBordoni added this to the Export Modul milestone Nov 6, 2024
@MiraGeowerkstatt
Copy link
Contributor

In Absprache mit Michael per Teams können die csv header im Camel Case direkt auf dem .Net api exportiert werden.
Im Vergleich zum jetzigen Import sollen folgende Änderungen gemacht werden:

QT_Reference_Elevation_Id = ReferenceElevationPrecisionId
Qt_Depth_Id = DepthPresicionId
Chronostratigraphie_id =ChronostratigraphieTopBedrockId
Lithostratigraphie_id = LithostratigraphieTopBedrockId
alternate_name = Name

NationalInterest hinzufügen.

@gysimichael
Copy link
Contributor

OriginalId und KantonId sollen importiert werden können -> geht noch nicht. Export gibt sie mit, aber beim Import kommen sie nicht mit

Niederlenz-1-4.csv

@gysimichael
Copy link
Contributor

rest geht einwandfrei, abgesehen vom Bug #1781 mit der Visualisierung. Wenn man aber die importierte Bohrung wieder Export, sind die 2 Werte da, sprich ist nur ein UI Problem

@MiraGeowerkstatt
Copy link
Contributor

OriginalId und KantonId sollen importiert werden können -> geht noch nicht. Export gibt sie mit, aber beim Import kommen sie nicht mit

Niederlenz-1-4.csv

Oh das ist genau so ein Fall warum wir nicht so happy waren mit dem Import/Export der custom Identifiers.
Wir erstellen die Spalten-Header aus der englischen Übersetzung der entsprechenden Codelist. Auf DEV scheint die Übersetzung mal geändert zu haben. Auf einer anderen Instanz kann es natürlich theoretisch auch nochmal ganz anders sein. Wird im csv file OriginalID auf IDOriginal geändert funktioniert der Import . Es ist aber natürlich unglücklich, da der Export den Spaltenheader OriginalID schreibt. Ich schaue mal noch ob ich da eine Lösung finde. Ansonsten müssen wir das im Januar nochmal genau anschauen.

Objekt auf DEV : {
"id": 100000004,
"geolcode": 100000004,
"schema": "borehole_identifier",
"code": "100000004",
"en": "Original ID",
"de": "ID Original",
"fr": "ID Originale",
"it": "ID Original",
"ro": null,
"order": 1,
"conf": null,
"isDefault": false,
"path": null
},

Bei uns lokal

{
"id": 100000004,
"geolcode": 100000004,
"schema": "borehole_identifier",
"code": "100000004",
"en": ID Original",
"de": "ID Original",
"fr": "ID Originale",
"it": "ID Original",
"ro": null,
"order": 1,
"conf": null,
"isDefault": false,
"path": null
},

@gysimichael
Copy link
Contributor

Verstehe; könnest du bitte prüfen, ob die IDs bei uns bei allen Instanzen gleich sind?

@gysimichael
Copy link
Contributor

ah, und help müsste man wohl auch noch anpassen

Image

da geht nun ja nicht mehr alles

@gysimichael
Copy link
Contributor

ja, mit dem richtigen heade geht es

@gysimichael
Copy link
Contributor

Somit müsste man den Export auf IDOriginal und IDCanton ändern, korrekt?

@MiraGeowerkstatt
Copy link
Contributor

MiraGeowerkstatt commented Dec 20, 2024

Ein Quickfix wäre es bei uns die Codelist Übersetzung der Übersetzung auf DEV anzugleichen. Ist aber eben dann nur so lang stabil wie da nichts ändert. Die Help stimmt weiterhin, da wir alles drin gelassen haben was schon war. Kann also alles noch importiert werden. Der Export generiert die Header dynamisch aufs den Codelists, vielleicht liesse sich für den Import ein vergleichbarer dynamischer Mechanismus finden.

@gysimichael
Copy link
Contributor

also auch die anderen ID lassen sich somit importieren?

@gysimichael
Copy link
Contributor

gut, im UI kann man die IDs nicht mehr Manuel ändern... das hilft sicher

@MiraGeowerkstatt
Copy link
Contributor

also auch die anderen ID lassen sich somit importieren?

Ja sofern die Header denen in der Help dokumentierten entsprechen.

@gysimichael
Copy link
Contributor

ja, auch geoMol kommt mit, wenn man den Exportheader ändert

@gysimichael
Copy link
Contributor

somit wäre gut, wenn beim Export das gleich bezüglich ID header rauskommt, wie beim Import gefordert, sprich, überall das ID vorausnehmen

@MiraGeowerkstatt
Copy link
Contributor

gut, im UI kann man die IDs nicht mehr Manuel ändern... das hilft sicher

Löschen und Neuerfassen wäre der workflow dafür, ja

@gysimichael
Copy link
Contributor

ich meine im Setup, war ja füher möglich

@MiraGeowerkstatt MiraGeowerkstatt linked a pull request Dec 20, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants