Project

General

Profile

バグ #1343

Mastodonプラグインが使っているactivityタイプを、専用のものにするべき

Added by toshi_a 初音 6 months ago. Updated 6 months ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
ブランチ:
topic/1343-mastodon-activity-kind
クラッシュする:
No

Description

Mastodonプラグインが発生させるアクティビティメッセージのうち、systemタイプになっているものを別のタイプに修正する。

現在は全てがsystemになっており、Mastodonに関する通知をユーザが制御できない状態になっている


Files

0001-mastodon-system.patch (2.1 KB) 0001-mastodon-system.patch Shibafu Midorino, 2019-06-08 15:02

Associated revisions

Revision 6859b92b (diff)
Added by Shibafu Midorino 6 months ago

フォロー関係の取得通知をsystemから隔離

refs #1343

Revision 7fc49f19 (diff)
Added by toshi_a 初音 6 months ago

不要なactivity通知の削除 refs #1343

History

#1

Updated by Shibafu Midorino 6 months ago

現在 system activity が使用されているのは3箇所でしょうか。

[shibafu@lulua ~/git/mikutter]$ rg ":system" core/plugin/mastodon
core/plugin/mastodon/api.rb
124:          Plugin.activity(:system, "APIアクセス失敗", description: "URI: #{uri}\nparameters: #{params}\nHTTP status: #{resp.status}\nresponse:\n#{resp.body}") rescue nil

core/plugin/mastodon/mastodon.rb
49:    activity(:system, "自分のプロフィールやフォロー関係を取得しています...")
56:        activity(:system, "自分のプロフィールやフォロー関係の取得が完了しました(#{world.account.acct})")

そのうち後者については、worldon時代に個人的に使用していたパッチがありますので提供します。

前者は、このようなケースの場合ってactivityで出すべきなんでしょうかね…?

#2

Updated by toshi_a 初音 6 months ago

  • Assignee set to toshi_a 初音
  • ブランチ set to topic/1343-mastodon-activity-kind

ブランチ作成しましたが、俺も変更入れるのでもう少し待ってください(後述)

前者は、このようなケースの場合ってactivityで出すべきなんでしょうかね…?

いえ、ないです。これを消すのは俺の方でやります。
初期のmikutterはTwitterがレスポンスを返してくることは稀であるという考え方で一切が設計されているからです。

例外として、ユーザの操作によって能動的に取得される場合は失敗時のフィードバックがあったほうが良いですが、APIリクエストを行うレイヤーではこの判断は出来ません。

#3

Updated by toshi_a 初音 6 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Shibafu Midorino

対応しました。確認してもらって良いですか。

#4

Updated by Shibafu Midorino 6 months ago

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

確認しました。問題なく動作しています。

#5

Updated by toshi_a 初音 6 months ago

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

merged.

Also available in: Atom PDF