search

lucascranach / importer

MIT License
1 stars 1 forks source link

JSON Url addieren? #11

Open cnoss opened 3 years ago

cnoss commented 3 years ago

Wäre vielleicht praktisch/ sinnvoll. Aber nicht zwingend erforderlich :)

Bildschirmfoto 2021-04-05 um 20 17 16
jhfpereira commented 3 years ago

Wie besprochen, würde ich anstatt eines dataJsonUrl-Feld, Felder für die Werte imageserver-2021 (Ordner) und 1.1 (Version) einführen, damit die URL zum imageData-JSON dynamisch konstruiert werden kann.

jhfpereira commented 3 years ago

Habe es mir nochmal angeschaut. Es hätte Sinn gemacht imageserve-2021 und 1.1 in ein jeweils eigenes Feld abzulegen. Wenn man aber bedenkt, dass Bild-URLs aktuell ebenfalls absolut inkl. Angabe des Protokolls und des Hosts enthalten, würde man an zwei Stellen verschiedene Ansätze fahren.

Würde deshalb im ersten Schritt für das imageData-File ebenfalls die komplette URL in einem Feld mit dem Namen dataJsonSrc hinterlegen, wie vorgeschlagen. Zu einem späteren Zeitpunkt könnte man die Protokoll- und Host-Angaben in allen URLs in einem Schritt anders handhaben.

jhfpereira commented 3 years ago

Nachdem ich ein wenig am PR bezüglich Viewer im Gatsby-Projekt saß, scheint mir die Existenz eines dataJsonURL-Felds jetzt nicht mehr als notwendig. Glaube dieser war glaube ich auch nur dafür gedacht, um frontend-seitig leichter asynchron an die restlichen Bilder zu kommen. Da im Rahmen des PRs nun alle Bilder zu einem Objekt zur Buildtime vorliegen, muss nicht mehr der Inhalt einer imageData-1.1.json ausgelesen werden, wodurch sich Requests erübrigen und somit auch die Angabe der URL zur imageData-1.1-json in den Daten.