merefield
13 Febbraio 2024, 8:10pm
1
Potrei starmi sbagliando, ma questo componente si aspetta un modello con un oggetto topic contenuto:
Tuttavia, quando registro questo modello, non c’è più un attributo topic?:
C’è un motivo per cui l’attributo topic è stato rimosso? Lo stavo effettivamente usando anche per un’estensione dell’adplugin
2 Mi Piace
david
14 Febbraio 2024, 11:45am
3
Ciao Robert, non credo che fosse intenzionale. Sei riuscito a scoprire a cosa si riferisce @model? Sarebbe utile capire se si è trattato di una modifica al core o a un plugin.
1 Mi Piace
merefield
14 Febbraio 2024, 11:56am
4
Non sta prendendo il modello post passato dal connettore post-bottom?
{{post-bottom-ad model=this}}
Detto questo, ho cercato post-bottom nel repository di discourse e non è emerso nulla… scaverò un po’ di più.
1 Mi Piace
merefield
14 Febbraio 2024, 12:44pm
5
Questo sembra codice molto vecchio:
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 }
Che contiene l’oggetto topic, quindi le cose cambiano sotto questo
2 Mi Piace
david
14 Febbraio 2024, 1:43pm
6
Ok, ho capito cosa è successo qui: grazie per l’avviso @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.
Due cose:
Durante alcune refactoring qualche mese fa, abbiamo iniziato inavvertitamente a passare il ‘transformedPost’ invece di ‘post’ a post-bottom-ad
Durante la stessa refactoring, abbiamo lasciato in giro un vecchio file post-bottom/discourse-adplugin.hbs. Non faceva nulla, ma ha reso questo problema molto più difficile da capire
4 Mi Piace
merefield
14 Febbraio 2024, 2:22pm
7
David Taylor:
difficile da capire
È la descrizione perfetta
L’architettura dell’interfaccia utente dello stream dei post è piuttosto complessa!
Grazie mille per aver esaminato questo!!
6 Mi Piace
david
19 Febbraio 2024, 8:00am
8
Questo argomento è stato chiuso automaticamente dopo 4 giorni. Non sono più ammessi nuovi messaggi.
