Project

General

Profile

Actions

バグ #1445

closed

boostされた時にMention通知の音が鳴る

Added by Shibafu Midorino over 4 years ago. Updated about 4 years ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
クラッシュする:
No

Description

気づいた範囲では 19a54a3e から 4bf92203 にかけて、boostされた時にMention通知も発生しているように見えます。少なくとも通知音はそのようになっています。


再現手順

1. ウケそうなことを書く
2. 誰かにboostされる

Actions #2

Updated by toshi_a 初音 over 4 years ago

  • Status changed from 分類待ち to 実装待ち

Twitterではもともと、「RT <screen_name>: <original content>」という書式でツイートすることでリツイートを表現していましたが、これは<screen_name>が含まれているのでmentionとなっていました。その後公式にリツイートが実装された後も、リツイートの本文は上記のようになっていたため、自分のツイートを誰かにリツイートされると、それはmentionイベントに含まれました。リツイートを通知するには、mentionイベントからリツイートだけを取り出して反応してやることになっていました。

mentionイベントはTwitterのmentionを表現するためのもので、互換性のために残っているものを未だに利用しているものなので、引き続きこの制約に従わなければなりません。

現在ではshareイベントがあって、これをhookすれば期待通りになると思うので(sub_parts_shareがこれを使っている)、その方向で考えてみようかな

Actions #3

Updated by Shibafu Midorino over 4 years ago

  • Status changed from 実装待ち to レビュー待ち
  • ブランチ set to topic/1445-mention-notify-cond

notifyプラグインのmention通知条件が怪しいので変更してみました。git log見てみたら数rev前はこうだったっぽい……?

Actions #4

Updated by Izumi Tsutsui about 4 years ago

  • Status changed from レビュー待ち to マージ待ち

ウケそうなことは書けませんでしたが再現したので
ce1b95c9 相当を 4.1.0-alpha3 に当ててみて
直ってるっぽいのを確認しました

Actions #5

Updated by toshi_a 初音 about 4 years ago

  • Status changed from マージ待ち to 終了

merged

Actions

Also available in: Atom PDF