merefield
(Robert)
13. Februar 2024 um 20:10
1
Ich bilde mir das vielleicht ein, aber diese Komponente erwartet ein Model mit einem enthaltenen Topic-Objekt:
Wenn ich dieses Model jedoch auslogge, gibt es kein Topic-Attribut mehr?:
Gibt es einen Grund, warum das Topic-Attribut entfernt wurde? Ich habe es tatsächlich auch für eine Erweiterung des Adplugins verwendet
2 „Gefällt mir“
david
(David Taylor)
14. Februar 2024 um 11:45
3
Hallo Robert, ich glaube nicht, dass das beabsichtigt war. Konntest du herausfinden, worauf sich @model bezieht? Es wäre gut zu klären, ob es sich um eine Kern- oder Plugin-Änderung handelte.
1 „Gefällt mir“
merefield
(Robert)
14. Februar 2024 um 11:56
4
Nimmt es nicht das Post-Modell, das vom post-bottom-Connector übergeben wird?
{{post-bottom-ad model=this}}
Dennoch habe ich eine Suche nach post-bottom im Discourse-Repository durchgeführt und es wurde nichts gefunden … ich werde noch etwas weiter graben.
1 „Gefällt mir“
merefield
(Robert)
14. Februar 2024 um 12:44
5
Dieser Code scheint sehr alt zu sein:
transformed.cloaked = _cloaked[post.id];
postTransformCallbacks(transformed);
if (transformed.isSmallAction) {
result.push(
this.attach("post-small-action", transformed, { model: post })
);
} else {
transformed.showReadIndicator = attrs.showReadIndicator;
result.push(this.attach("post", transformed, { model: post }));
}
// Post gap - after
const afterGap = after[post.id];
if (afterGap) {
result.push(
this.attach(
"post-gap",
{ pos: "after", postId: post.id, gap: afterGap },
{ model: post }
Der das Topic-Objekt enthält, also Dinge sich darunter ändern
2 „Gefällt mir“
david
(David Taylor)
14. Februar 2024 um 13:43
6
Ok, ich sehe, was hier passiert ist – danke für den Hinweis @merefield
main ← widget-model
opened 01:41PM - 14 Feb 24 UTC
Changes in 0948d6b19e34320f95226cc43fbce976b71ceb31 caused the post-bottom-ad co… mponent to start receiving the widget-ised 'transformedPost' instead of the original `post` model. In most cases this difference didn't matter, but it did cause noticable issues with `@model.category` and `@model.topic`.
This commit also deletes the unused `post-bottom/discourse-adplugin.hbs` connector. In the past, the adplugin itself was defining the post-bottom outlet. But now, we use RenderGlimmer to load the `post-bottom-ad` directly.
Zwei Dinge:
Während einiger Refactorings vor ein paar Monaten haben wir versehentlich ‘transformedPost’ anstelle von ‘post’ an post-bottom-ad übergeben.
Während desselben Refactorings haben wir eine alte post-bottom/discourse-adplugin.hbs-Datei herumliegen lassen. Sie hat zwar nichts bewirkt, aber sie hat dieses Problem waaay schwieriger zu lösen gemacht
4 „Gefällt mir“
merefield
(Robert)
14. Februar 2024 um 14:22
7
David Taylor:
schwierig herauszufinden
Das ist die perfekte Beschreibung
Die UI-Architektur des Post-Streams ist ziemlich gewaltig!
Vielen Dank, dass Sie sich damit beschäftigt haben!!
6 „Gefällt mir“
david
(David Taylor)
Geschlossen,
19. Februar 2024 um 08:00
8
Dieses Thema wurde nach 4 Tagen automatisch geschlossen. Neue Antworten sind nicht mehr möglich.