プロジェクト

全般

プロフィール

バグ #1343

完了

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

toshi_a 初音 さんがほぼ5年前に追加. ほぼ5年前に更新.

ステータス:
終了
優先度:
通常
対象バージョン:
プラグイン名:
クラッシュする:
いいえ

説明

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

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


ファイル

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

Shibafu Midorino さんがほぼ5年前に更新

現在 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で出すべきなんでしょうかね…?

toshi_a 初音 さんがほぼ5年前に更新

  • 担当者toshi_a 初音 にセット
  • ブランチtopic/1343-mastodon-activity-kind にセット

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

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

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

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

toshi_a 初音 さんがほぼ5年前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から Shibafu Midorino に変更

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

Shibafu Midorino さんがほぼ5年前に更新

  • ステータスレビュー待ち から マージ待ち に変更

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

toshi_a 初音 さんがほぼ5年前に更新

  • ステータスマージ待ち から 終了 に変更

merged.

他の形式にエクスポート: Atom PDF