Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: タブ設定に「全タブのメッセージを消去」を追加 #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NktAts
Copy link
Contributor

@NktAts NktAts commented Dec 1, 2019

概要

 タブ設定画面の最下部に、「全タブのメッセージを消去」ボタンを追加しました。
 押下すると、タブの項目はそのままに、すべてのメッセージを消去します。

更新理由

 多人数で1つのルームにログイン、かつメッセージの数が大量になった場合(確認した範囲では6人以上かつ全メッセージ5000以上)、ログインが行えなくなる事象を確認しました。
 chat.xmlをいじる、全タブ消去ののち作り直すなどは手間だったため、メッセージ消去ボタンを追加しました。

実装詳細

 実装上は、「各タブを一度destroy」→「同じ名称で再度生成」という形で実装しています。

@zeteticl
Copy link
Contributor

This is easy to trigger by mistake. Hope to add a confirmation popup.

usaturn pushed a commit to usaturn/udonarium that referenced this pull request Apr 4, 2022
…gration

デバッグ用のsourcemapがng serveで生成されるようangular.jsonを変更
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants