merefield
Fevereiro 13, 2024, 8:10pm
1
Posso estar vendo coisas, mas este componente espera um modelo com um objeto de tópico contido:
No entanto, quando faço o log deste modelo, não há mais um atributo de tópico?:
Existe algum motivo para o atributo de tópico ter sido removido? Eu estava realmente usando-o também para uma extensão do adplugin
2 curtidas
david
Fevereiro 14, 2024, 11:45am
3
Oi Robert - Não acho que isso tenha sido intencional. Você conseguiu descobrir a que @model se refere? Seria bom descobrir se foi uma alteração principal ou de plugin.
1 curtida
merefield
Fevereiro 14, 2024, 11:56am
4
Não está a receber o modelo post passado pelo conector post-bottom?
{{post-bottom-ad model=this}}
Dito isto, realizei uma pesquisa por post-bottom no repositório do discourse e não encontrei nada… Vou investigar um pouco mais.
1 curtida
merefield
Fevereiro 14, 2024, 12:44pm
5
Este parece ser um código muito antigo:
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 }
Que contém o objeto do tópico, então as coisas mudam abaixo disso
2 curtidas
david
Fevereiro 14, 2024, 1:43pm
6
Ok, entendi o que aconteceu aqui - obrigado pelo aviso @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.
Duas coisas:
Durante uma refatoração há alguns meses, inadvertidamente começamos a passar o ‘transformedPost’ em vez de ‘post’ para o post-bottom-ad
Durante a mesma refatoração, deixamos um arquivo antigo post-bottom/discourse-adplugin.hbs por aí. Ele não fazia nada de fato, mas tornou este problema bem mais complicado de descobrir
4 curtidas
merefield
Fevereiro 14, 2024, 2:22pm
7
David Taylor:
difícil de entender
É a descrição perfeita
A arquitetura da interface do usuário do fluxo de postagens é bem complexa!
Muito obrigado por investigar isso!!
6 curtidas
david
Fevereiro 19, 2024, 8:00am
8
Este tópico foi automaticamente fechado após 4 dias. Novas respostas não são mais permitidas.
