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

/channels/{channelId}/statsに削除済みのメッセージも集計されているのが分かりづらい #2357

Open
oribe1115 opened this issue Mar 31, 2024 · 0 comments
Labels
kind/enhancement 機能改善に関するもの

Comments

@oribe1115
Copy link

現在、削除済みのメッセージも含めた統計値を取得していそう
(traQのメッセージはsoft deleteなはずなので)

var stats repository.ChannelStats
if err := repo.db.Unscoped().
Model(&model.Message{}).
Select("COUNT(channel_id) AS total_message_count").
Where(&model.Message{ChannelID: channelID}).
Find(&stats.TotalMessageCount).
Error; err != nil {
return nil, err
}

現状のAPIドキュメントからはこのAPIの値が「削除済みメッセージも含んでいる」というのは非自明に思える
また、「削除されていないメッセージ(=現在閲覧可能)」の統計値が欲しいケースがありそうなのでクエリパラメータなどで含めるかどうかを切り替えられると嬉しそう

@motoki317 motoki317 added the kind/enhancement 機能改善に関するもの label May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement 機能改善に関するもの
Projects
None yet
Development

No branches or pull requests

2 participants