search

cloudnativedaysjp / dreamkast-ui

MIT License
7 stars 2 forks source link

fix: Add missing props of ChatBox to enable Chat deletion #302

Closed hrk091 closed 2 years ago

hrk091 commented 2 years ago

ChatBoxのevent propsに値が設定されておらず、結果としてchat削除時のガード節を突破できず削除ができない状態になっていたので、eventを追加しました。

  const onMenuClick = (e: React.MouseEvent<HTMLElement>) => {
    const selectedMessageId = e.currentTarget.getAttribute('data-messageId')
    if (!selectedMessageId || !event) return  // <- ここでこけてしまいます
    const api = new ChatMessageApi(
      new Configuration({ basePath: window.location.origin }),
    )
    const newChatMessage = {
      eventAbbr: event.abbr,
      body: 'このメッセージは削除されました',
    }
    api.apiV1ChatMessagesMessageIdPut(selectedMessageId, newChatMessage)
    setAnchorEl(null)
  }

特にconsole.errorをはいたりthrowしたりしないので、サイレントな不具合になっていたように見えます。 一通り動作確認して動いていそうではありますが、コードを全て追いかけることはできていないので、まずいケースがあればご指摘いただけると助かります（eventが設定されている方が正しい状態に見えるので、大丈夫だとは思うのですが）

github-actions[bot] commented 2 years ago

Review app