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

MAINT: Let TMSGallery not implement Gallery #838

Closed
wants to merge 1 commit into from

Conversation

adrian-zon
Copy link
Contributor

Dieser PR ist erstmal nur ein Diskussionsbeitrag :) TMSGallery gibt gar keine Bilder zurück, was ich recht irreführend finde. Am liebsten wäre mir, wenn das Objekt mit dem wir es in Teasern für Galerien zu tun haben in beiden Fällen nur unterstützen würde was TMSGallery kann, aber das geht vermutlich nicht.

Checklist

  • Documentation
  • Changelog
  • Tests
  • Translations

gif

Wheel Of Fortune Idk - Sm6I4hAtuREavkhxJy

TMSGallery doesn't actually provide the gallery images.
@wosc
Copy link
Member

wosc commented Aug 29, 2024

Ich vermute, der Fall "Gallerie in Autofläche" ist ziemlich selten, jedenfalls wenn ich mir den Code so anschaue, würde ich sagen, das kann noch nie funktioniert haben: Die Gallerie-Einträge stehen im XML-Body, diese TMS-Proxies basteln sich dafür aber einen Fake zusammen, der nur die paar Dinge enthält, die im TMS-Result halt drinstehen (title und sowas, vgl übrigens auch ZO-788). Das kann also so nix werden. Da wir grade aber ja eh daran arbeiten, die Autoflächen direkt aus dem Content-Storage zu bestücken, würde ich hier keinen großen Aufwand mehr investieren.

Wenn das im Zusammenspiel mit zeit.web dann besser aussehende Teaser produziert, indem man hier als TMSGallery ein Objekt rausgibt, was überhaupt keine Gallery ist, können wir das als Workaround meinetwegen gern so machen.

@adrian-zon
Copy link
Contributor Author

https://www.zeit.de/foto/index ist der klassische Fall. Wenn das ausm Content-Storage kommt, kriege ich dann ein richtiges Gallery-Objekt? Dann lohnt es glaub ich nicht mehr, diese Frage weiter zu erörtern.

@wosc
Copy link
Member

wosc commented Aug 29, 2024

Ja genau, diese ganzen TMS-Proxies sind eine reine Performance-Optimierung für ES/TMS-basierte Autoflächen; wenn die Autoflächen direkt aus dem Content-Storage bestückt werden, wird man dann ganz normale Objekte bekommen.

@wosc wosc closed this Aug 29, 2024
@wosc wosc deleted the maint/tms_gallery branch August 29, 2024 14:16
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

Successfully merging this pull request may close these issues.

2 participants