Link do chat ausente no widget de evento de postagem após o Rollup

Olá :waving_hand:

Encontramos um pequeno bug na funcionalidade de chat do plugin discourse-calendar, onde o link :speech_balloon: para o canal de chat desapareceu repentinamente no widget de evento da postagem.

Muito provavelmente, isso foi quebrado pela recente mudança no núcleo do Discourse para migrar o sistema de build para Rollup:

  • af3385baba — Introdução do compilador Rollup (desativado por padrão)
  • 74d532c559 — Ativação do compilador Rollup como padrão

Posso corrigir isso e fornecer um PR para tal :technologist:.

Comportamento esperado

Ao adicionar um evento a um tópico e habilitar a Integração de Chat (“Criar e gerenciar canal de chat específico para o evento”), o widget de evento na postagem incluiria um ícone :speech_balloon: com um link para o canal de chat associado ao evento:

Comportamento atual

No momento, na main atual, essa seção do widget de evento da postagem simplesmente não é renderizada:

Correção/Solução

O problema parece ser que o helper optionalRequire preguiçoso (lazy) não funciona mais como inicialmente pretendido com o Rollup. Pelo que entendi, o Rollup chama o optionalRequire muito cedo, antes que outros módulos estejam registrados, fazendo com que ele sempre retorne false.

O que resolve o problema é mover as chamadas de optionalRequire para o construtor do componente, para que sejam realmente avaliadas de forma preguiçosa (lazy).

Para o arquivo de modelo, por exemplo, ficaria assim:

diff --git a/plugins/discourse-calendar/assets/javascripts/discourse/models/discourse-post-event-event.js b/plugins/discourse-calendar/assets/javascripts/discourse/models/discourse-post-event-event.js
index 742cc35a84..f687b956aa 100644
--- a/plugins/discourse-calendar/assets/javascripts/discourse/models/discourse-post-event-event.js
+++ b/plugins/discourse-calendar/assets/javascripts/discourse/models/discourse-post-event-event.js
@@ -7,10 +7,6 @@ import User from "discourse/models/user";
 import DiscoursePostEventEventStats from "./discourse-post-event-event-stats";
 import DiscoursePostEventInvitee from "./discourse-post-event-invitee";
 
-const ChatChannel = optionalRequire(
-  "discourse/plugins/chat/discourse/models/chat-channel"
-);
-
 const DEFAULT_REMINDER = {
   type: "notification",
   value: 15,
@@ -62,6 +58,8 @@ export default class DiscoursePostEventEvent {
   @tracked _reminders;
 
   constructor(args = {}) {
+    const ChatChannel = optionalRequire( "discourse/plugins/chat/discourse/models/chat-channel");
+
     this.id = args.id;
     this.rrule = args.rrule;
     this.name = args.name;

E respectivamente para o componente que renderiza o link real:

diff --git a/plugins/discourse-calendar/assets/javascripts/discourse/components/discourse-post-event/chat-channel.gjs b/plugins/discourse-calendar/assets/javascripts/discourse/components/discourse-post-event/chat-channel.gjs
index f673ee8a9f..a988b341e6 100644
--- a/plugins/discourse-calendar/assets/javascripts/discourse/components/discourse-post-event/chat-channel.gjs
+++ b/plugins/discourse-calendar/assets/javascripts/discourse/components/discourse-post-event/chat-channel.gjs
@@ -1,24 +1,27 @@
+import Component from "@glimmer/component";
 import { LinkTo } from "@ember/routing";
 import { optionalRequire } from "discourse/lib/utilities";
 import { and } from "discourse/truth-helpers";
 
-const ChannelTitle = optionalRequire(
-  "discourse/plugins/chat/discourse/components/channel-title"
-);
+export default class DiscoursePostEventChatChannel extends Component {
+  get channelTitle() {
+    return optionalRequire(
+      "discourse/plugins/chat/discourse/components/channel-title"
+    );
+  }
 
-const DiscoursePostEventChatChannel = <template>
-  {{#if (and @event.channel ChannelTitle)}}
-    <section class="event__section event-chat-channel">
-      <span></span>
-      <LinkTo
-        @route="chat.channel"
-        @models={{@event.channel.routeModels}}
-        class="chat-channel-link"
-      >
-        <ChannelTitle @channel={{@event.channel}} />
-      </LinkTo>
-    </section>
-  {{/if}}
-</template>;
-
-export default DiscoursePostEventChatChannel;
+  <template>
+    {{#if (and @event.channel this.channelTitle)}}
+      <section class="event__section event-chat-channel">
+        <span></span>
+        <LinkTo
+          @route="chat.channel"
+          @models={{@event.channel.routeModels}}
+          class="chat-channel-link"
+        >
+          <this.channelTitle @channel={{@event.channel}} />
+        </LinkTo>
+      </section>
+    {{/if}}
+  </template>
+}

(Nota: Preciso convertê-lo para um componente de classe, por isso o diff é maior)

Vou fornecer um PR com minha correção no GitHub, para que as pessoas possam testar e expressar suas opiniões sobre isso.

2 curtidas

Obrigado @klappradla. Sua correção proposta parece boa para mim - por favor, poste um link para o PR assim que estiver pronto :rocket:

Aqui está → https://github.com/discourse/discourse/pull/39287

1 curtida