Project

General

Profile

Actions

バグ #1343

closed

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

Added by toshi_a 初音 over 5 years ago. Updated over 5 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
クラッシュする:
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
Actions #1

Updated by Shibafu Midorino over 5 years 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で出すべきなんでしょうかね…?

Actions #2

Updated by toshi_a 初音 over 5 years ago

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

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

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

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

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

Actions #3

Updated by toshi_a 初音 over 5 years ago

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

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

Actions #4

Updated by Shibafu Midorino over 5 years ago

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

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

Actions #5

Updated by toshi_a 初音 over 5 years ago

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

merged.

Actions

Also available in: Atom PDF