Closed tbaddade closed 3 years ago
Gehört für mich eher in eine 3.1.1, da ohne diese Änderungen aktuell kein Streamabruf mehr möglich ist.
3.3 kann man gar nicht auswählen https://github.com/FriendsOfREDAXO/feeds/blob/a6c84cfa27f8f4cabf4193e36110b15bf3472ad3/lib/stream/facebook_feed.php#L55
3.3 kann man gar nicht auswählen
Ich weiß nicht was diese Auswahl genau für eine Auswirkung hat, aber die App selber ist mit v6 bei Facebook angelegt. Ein Downgrade ist dort nicht möglich. D.h. ich habe in Feed v3.2 ausgewählt und den Stream abgerufen. Da v6 diese Felder nicht mehr liefert, kam dann dieser Whoops zustande.
OK, das wäre dann aber nicht mehr BC, die Felder einfach zu entfernen.
Normalerweise sagt die Auswahl, mit welcher Version eben kommuniziert werden soll, sofern das die App zulässt.
Wir können aber gerne die 6.0 zur Auswahl stellen oder – falls wirklich nicht mehr benötigt – auf die Auswahl zukünftig verzichten, wenn die App ohnehin die Vorgabe liefert.
Dann wäre es gut, wenn im PR die 3.3-6.0 zur Auswahl stehen und ab 3.3 die Abfrage geändert wird.
Dann können wir es in Feeds 3.1.1 direkt aufnehmen.
Generell sollten auch hier Streams in Feeds verbessert werden, dass solche Fehler nicht zu einem Whoops führen. Ich habe in vergangenen PRs dazu einiges gemacht, aber hier sollte man den Fehler catchen.
Wenn die Auswahl der API-Version keinen Einfluss hat, braucht es auch keine Ergänzung von den fehlenden Versionen. Wenn die Auswahl bleibt, dann müsste m.E. ein Fehler geworfen werden, sobald der Stream und die App unterschiedliche Versionen haben.
Wie ich eben sehe, hast sogar du die Auswahl ergänzt :). Vielleicht könntest du das besser beurteilen.
Die Auswahl der API-Version kann nicht die mindest API-Version der App überschreiben.
Wie oben beschrieben wäre es am besten, wenn die Auswahl entweder ganz verschwindet - dann müsste Feeds aber immer sicherstellen, dass die angeforderten Felder auch geliefert werden - oder man behält die Auswahl und entscheide dann, welche Felder abgefragt werden.
Ich wäre dafür, die Auswahl zu behalten, um in Feeds abwärtskompatibel zu bleiben und stattdessen ab 3.3 die Felder nicht mehr abzufragen.
Das Anpassen der Versionsauswahl sollte eher in einem separaten PR einfliessen. Ob die Korrekturen hier mit der aktuellen Version von FB noch gültig sind, kann ich nicht beurteilen. Auch ab welcher Version diese Felder nicht mehr existieren weiß ich nicht und kann eine BC nicht umsetzen.
Magst du deshalb den PR so übernehmen wie er ist, gern anpassen oder einfach schließen?
Ich habe es auf dem Schirm. Vermutlich wird dein Fix Ende des Monats auch nicht mehr funktionieren. Deswegen warte ich noch ab...
Sonst gibt's nur Support-Anfragen, warum es 3 Tage funktionert und dann wieder nicht.
Wie würdest du das zukünftig lösen und auf welchen Aufwand schätzt du das?
Wie würdest du das zukünftig lösen
Ich würde nicht versuchen, ältere App-Versionen mit durchzuschleifen sondern mich nur an der aktuellsten halten. Diese Abhängigkeit irgendwo hinterlegen. Wer Feed nutzt muss dann sicherstellen, dass auch seine App mit der neusten API funktioniert. Ansonsten das Update von Feed auslassen.
und auf welchen Aufwand schätzt du das?
Es war ein Grund, wegen dieser ständigen Aktualisierungen (hier war insbesondere Instagram so ein Kandidat), dass YFeed bei FOR gelandet ist. Hier sollten sich mehrere Leute beteiligen und versuchen das AddOn am Laufen zu halten. Allein würde ich das nicht betreuen wollen. Mehr kann ich zum Aufwand nicht sagen, wer weiß was da in Zukunft seitens der Anbieter passiert.
Funktionert dein Fix aktuell noch?
Funktionert dein Fix aktuell noch?
Das Projekt wo ich den Fix eingebaut hatte, verwendet leider kein Feed mehr. Kann ich somit erst beim nächsten Projekt beurteilen oder jemand anderes.
Ich schließe, da niemand weiter diesen Fix bestätigen kann.
der Fix wurde inzwischen zu einem anderen Zeitpunkt übernommen, wie es scheint, deshalb lösche ich die Branch.
Bei Abruf des Streams kam folgende Meldung:
link field is deprecated for versions v3.3 and higher