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

Living Images Video-URL und Meta-Daten stehen im Backend zur Verfügung #2

Open
tjorbo opened this issue Nov 22, 2024 · 9 comments
Open
Assignees

Comments

@tjorbo
Copy link
Member

tjorbo commented Nov 22, 2024

Story

Als PV möchte ich, dass die Living Images auf Dataport-Infrastruktur bereit gestellt werden und über das Backend verknüpft sind um die Living Images zu Dataport zu migrieren.

Technische Details

Akzeptanzkriterien

  • die Video-URL ist im API-Endpunkt für Living Images hinterlegt
  • die Metadaten können von einem API-Entpunkt heruntergeladen werden
  • die Referezt auf das Living Image kann in der Institution hinterlegt werden
@tjorbo tjorbo moved this to 🆕 New in Kulturfinder Nov 22, 2024
@tjorbo
Copy link
Member Author

tjorbo commented Nov 22, 2024

@Winterhm könntest du dich um dieses Ticket kümmern?

@93Dennis
Copy link

93Dennis commented Dec 9, 2024

@Winterhm
Um fortzufahren, werden die Daten online in der API benötigt.

Bisher sieht die Datenarchitektur eines Living Images wie folgt aus:

  • id: string, must be unique
  • title: string, seen in overview
  • imageUrl: string, jp(e)g/png
  • setUrl: string, feature set name without extensions
  • videoUrl: string, mp4
  • width: number, video width
  • height: number, video height
  • scale: number, need to adjust this to make the video fit the marker image
  • offset = { x, y, z }: object with three numbers, to adjust video position

@tjorbo
Copy link
Member Author

tjorbo commented Dec 9, 2024

Unbenanntes Diagramm drawio(1)

Moin, so in etwa habe ich mir die Architektur vorgestellt. Gern könnt ihr ergänzen 👍

@tjorbo tjorbo moved this from 📋 Backlog to 🏗 In progress in Kulturfinder Dec 10, 2024
@93Dennis
Copy link

93Dennis commented Dec 11, 2024

@tjorbo Frage zu der Darstellung: Werden es zwei verschiedene APIs sein für Institution und LIs? Gern können die LIs direkt mit der Institution ausgeliefert werden.

@tjorbo
Copy link
Member Author

tjorbo commented Dec 13, 2024

Werden es zwei verschiedene APIs sein für Institution und LIs? Gern

@93Dennis Es sind zwei verschiedene API-Endpunkte.

Wir wollen die Auslieferung der Institution nicht um die Binärdaten der LIs vergrößern um die UX der "Hauptanwendung" Kulturfinder nicht durch zu große Netzwerkpakete zu gefähren.

Außerdem erhöht das die Testbarkeit und die unabhängige Entwicklung der beiden Anwendungen. Wir müssen also nur eine LivingImage ID zur lokalen Entwicklung verwenden und keine Institution.

@Winterhm
Copy link
Collaborator

@93Dennis Die API ist jetzt für die Living Images erweitert. Kannst du dir das ansehen und bescheid geben, ob das für dich so stimmt?

Der Endpunkt AddMediaToInstitute wurde erweitert, um auch die LIs anzulegen.
Die Institute Getter würden dann innerhalb des Arrays Media die LivingImageId liefern, die dann genutzt werden kann, um über GetLivingImage die Daten des LI zu holen.

@tjorbo tjorbo moved this from 🏗 In progress to 👀 In review in Kulturfinder Dec 17, 2024
@tjorbo
Copy link
Member Author

tjorbo commented Dec 18, 2024

@93Dennis Hier die Testinstanz: https://kulturfindertest.dsecurecloud.de/api/swagger/index.html

@Winterhm

1. Media Array

Können wir statt

{
        "id": 6,
        "mediaType": "Image",
        "alternateText": "Bild",
        "filename": "Storm Viola tricolorö.jpg",
        "copyright": "test",
        "artist": "test",
        "order": 0,
        "livingImageId": null
},

kein neuens Property ausgeben?

{
        "id": 6,
        "mediaType": "LivingImage",
        "alternateText": "Bild",
        "filename": "<ID>",
        "copyright": "test",
        "artist": "test",
        "order": 0,
},

2. Files

Können wir die fset Dateien und das Marker-Bild mit in den GetLivingImage Endpunkt unterbringen?

3. Testdatei

Magst du das Living Image in der Testinstanz einfügen? Welche Institution ist mir egal.

Danke 🍀

@Winterhm
Copy link
Collaborator

@tjorbo
zu 1. könnten wir natürlich machen. Einen Parameter zu zweckentfremden fände ich nur nicht so schön. Kann ich aber sonst machen.

zu 2. Für das MarkerBild gebe ich dann eine url mit.
Die fset Dateien würde ich dann noch über den "AddMediaToInstitute" Endpunkt hinzufügen lassen. Sind die immer wie im Beispiel "Jenny ENG" zusammengesetzt (*.fset, *.fset3, *.iset)?

@tjorbo
Copy link
Member Author

tjorbo commented Dec 20, 2024

@Winterhm
zu 1. Wir können gerne die ganze URL angeben. z.B. https://kulturfindertest.dsecurecloud.de/api/Media/GetLivingImage?id=1 das beugt auch Fehler zwischen prod und test instanzen vor.

zu 2. Ja genau, Meta-Dateien sind immer ein Bild (*.jpg, *.png), jeweilse eine Datei mit *.fset, *.fset3, *.iset

Danke Dir 🍀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants