バグ #1343
完了
Mastodonプラグインが使っているactivityタイプを、専用のものにするべき
toshi_a 初音 さんが5年以上前に追加.
5年以上前に更新.
説明
Mastodonプラグインが発生させるアクティビティメッセージのうち、systemタイプになっているものを別のタイプに修正する。
現在は全てがsystemになっており、Mastodonに関する通知をユーザが制御できない状態になっている
ファイル
現在 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 初音 にセット
- ブランチ を topic/1343-mastodon-activity-kind にセット
ブランチ作成しましたが、俺も変更入れるのでもう少し待ってください(後述)
前者は、このようなケースの場合ってactivityで出すべきなんでしょうかね…?
いえ、ないです。これを消すのは俺の方でやります。
初期のmikutterはTwitterがレスポンスを返してくることは稀であるという考え方で一切が設計されているからです。
例外として、ユーザの操作によって能動的に取得される場合は失敗時のフィードバックがあったほうが良いですが、APIリクエストを行うレイヤーではこの判断は出来ません。
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Shibafu Midorino に変更
- ステータス を レビュー待ち から マージ待ち に変更
他の形式にエクスポート: Atom
PDF